Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 906115 (CVE-2023-31038) - <dev-libs/log4cxx-1.2.0[odbc]: SQL injection
Summary: <dev-libs/log4cxx-1.2.0[odbc]: SQL injection
Status: CONFIRMED
Alias: CVE-2023-31038
Product: Gentoo Security
Classification: Unclassified
Component: Vulnerabilities (show other bugs)
Hardware: All Linux
: Normal minor
Assignee: Gentoo Security
URL: https://lists.apache.org/thread/vgjlp...
Whiteboard: B3 [cleanup glsa?]
Keywords: PullRequest
Depends on: 941223
Blocks:
  Show dependency tree
 
Reported: 2023-05-11 04:49 UTC by John Helmert III
Modified: 2024-10-11 22:05 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2023-05-11 04:49:55 UTC
CVE-2023-31038:

SQL injection in Log4cxx when using the ODBC appender to send log messages to a database.  No fields sent to the database were properly escaped for SQL injection.  This has been the case since at least version 0.9.0(released 2003-08-06)




Note that Log4cxx is a C++ framework, so only C++ applications are affected.

Before version 1.1.0, the ODBC appender was automatically part of Log4cxx if the library was found when compiling the library.  As of version 1.1.0, this must be both explicitly enabled in order to be compiled in.




Three preconditions must be met for this vulnerability to be possible:

1. Log4cxx compiled with ODBC support(before version 1.1.0, this was auto-detected at compile time)

2. ODBCAppender enabled for logging messages to, generally done via a config file

3. User input is logged at some point. If your application does not have user input, it is unlikely to be affected.





Users are recommended to upgrade to version 1.1.0 which properly binds the parameters to the SQL statement, or migrate to the new DBAppender class which supports an ODBC connection in addition to other databases. 
Note that this fix does require a configuration file update, as the old configuration files will not configure properly.  An example is shown below, and more information may be found in the Log4cxx documentation on the ODBCAppender.





Example of old configuration snippet:

<appender name="SqlODBCAppender" class="ODBCAppender">

    <param name="sql" value="INSERT INTO logs (message) VALUES ('%m')" />

    ... other params here ...

</appender>




The migrated configuration snippet with new ColumnMapping parameters:


<appender name="SqlODBCAppender" class="ODBCAppender">




    <param name="sql" value="INSERT INTO logs (message) VALUES (?)" />

    <param name="ColumnMapping" value="message"/>
    ... other params here ...


</appender>

Needs bump to 1.1.0.
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-01-21 04:45:43 UTC
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8790ed7ca284d11e3c0fbd5f2762f28dcac2d506

commit 8790ed7ca284d11e3c0fbd5f2762f28dcac2d506
Author:     Christopher Fore <csfore@posteo.net>
AuthorDate: 2024-01-20 16:17:09 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-01-21 04:26:24 +0000

    dev-libs/log4cxx: add 1.2.0, security bump
    
    Removed unnecessary prepare block now that the build system handles it
    properly.
    
    Bug: https://bugs.gentoo.org/908115
    Signed-off-by: Christopher Fore <csfore@posteo.net>
    Closes: https://github.com/gentoo/gentoo/pull/34686
    Signed-off-by: Sam James <sam@gentoo.org>

 dev-libs/log4cxx/Manifest             |  1 +
 dev-libs/log4cxx/log4cxx-1.2.0.ebuild | 45 +++++++++++++++++++++++++++++++++++
 2 files changed, 46 insertions(+)