mirror of
git://git.yoctoproject.org/poky.git
synced 2025-07-19 21:09:03 +02:00
libsoup-2.4: fix CVE-2025-32911
CVE-2025-32911: A use-after-free type vulnerability was found in libsoup, in the soup_message_headers_get_content_disposition() function. This flaw allows a malicious HTTP client to cause memory corruption in the libsoup server. Backport patches to fix it [1] https://nvd.nist.gov/vuln/detail/CVE-2025-32911 [2] https://gitlab.gnome.org/GNOME/libsoup/-/issues/433 (From OE-Core rev: 839d93bbb1ca7a51b659b8cb9def9b354a99518f) Signed-off-by: Changqing Li <changqing.li@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
6a17b9f352
commit
e5902fa07b
|
@ -0,0 +1,74 @@
|
||||||
|
From 52c5859b82fe79f2c32d883e048d218e0d7f2182 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Changqing Li <changqing.li@windriver.com>
|
||||||
|
Date: Wed, 30 Apr 2025 14:59:55 +0800
|
||||||
|
Subject: [PATCH] CVE-2025-32911
|
||||||
|
|
||||||
|
CVE: CVE-2025-32911
|
||||||
|
Upstream-Status: Backport [https://gitlab.gnome.org/GNOME/libsoup/-/merge_requests/422/commits]
|
||||||
|
|
||||||
|
Signed-off-by: Changqing Li <changqing.li@windriver.com>
|
||||||
|
---
|
||||||
|
libsoup/soup-message-headers.c | 13 +++++++++----
|
||||||
|
tests/header-parsing-test.c | 15 +++++++++++++++
|
||||||
|
2 files changed, 24 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/libsoup/soup-message-headers.c b/libsoup/soup-message-headers.c
|
||||||
|
index 39ad14a..78b2455 100644
|
||||||
|
--- a/libsoup/soup-message-headers.c
|
||||||
|
+++ b/libsoup/soup-message-headers.c
|
||||||
|
@@ -1454,10 +1454,15 @@ soup_message_headers_get_content_disposition (SoupMessageHeaders *hdrs,
|
||||||
|
*/
|
||||||
|
if (params && g_hash_table_lookup_extended (*params, "filename",
|
||||||
|
&orig_key, &orig_value)) {
|
||||||
|
- char *filename = strrchr (orig_value, '/');
|
||||||
|
-
|
||||||
|
- if (filename)
|
||||||
|
- g_hash_table_insert (*params, g_strdup (orig_key), filename + 1);
|
||||||
|
+ if (orig_value) {
|
||||||
|
+ char *filename = strrchr (orig_value, '/');
|
||||||
|
+
|
||||||
|
+ if (filename)
|
||||||
|
+ g_hash_table_insert (*params, g_strdup (orig_key), g_strdup(filename + 1));
|
||||||
|
+ } else {
|
||||||
|
+ /* filename with no value isn't valid. */
|
||||||
|
+ g_hash_table_remove (*params, "filename");
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
return TRUE;
|
||||||
|
}
|
||||||
|
diff --git a/tests/header-parsing-test.c b/tests/header-parsing-test.c
|
||||||
|
index 946f118..752196e 100644
|
||||||
|
--- a/tests/header-parsing-test.c
|
||||||
|
+++ b/tests/header-parsing-test.c
|
||||||
|
@@ -1034,6 +1034,7 @@ do_param_list_tests (void)
|
||||||
|
#define RFC5987_TEST_HEADER_FALLBACK "attachment; filename*=Unknown''t%FF%FF%FFst.txt; filename=\"test.txt\""
|
||||||
|
#define RFC5987_TEST_HEADER_NO_TYPE "filename=\"test.txt\""
|
||||||
|
#define RFC5987_TEST_HEADER_NO_TYPE_2 "filename=\"test.txt\"; foo=bar"
|
||||||
|
+#define RFC5987_TEST_HEADER_EMPTY_FILENAME ";filename"
|
||||||
|
|
||||||
|
static void
|
||||||
|
do_content_disposition_tests (void)
|
||||||
|
@@ -1133,6 +1134,20 @@ do_content_disposition_tests (void)
|
||||||
|
g_assert_cmpstr (filename, ==, RFC5987_TEST_FALLBACK_FILENAME);
|
||||||
|
parameter2 = g_hash_table_lookup (params, "foo");
|
||||||
|
g_assert_cmpstr (parameter2, ==, "bar");
|
||||||
|
+ g_hash_table_destroy (params);
|
||||||
|
+
|
||||||
|
+ /* Empty filename */
|
||||||
|
+ soup_message_headers_clear (hdrs);
|
||||||
|
+ soup_message_headers_append (hdrs, "Content-Disposition",
|
||||||
|
+ RFC5987_TEST_HEADER_EMPTY_FILENAME);
|
||||||
|
+ if (!soup_message_headers_get_content_disposition (hdrs,
|
||||||
|
+ &disposition,
|
||||||
|
+ ¶ms)) {
|
||||||
|
+ soup_test_assert (FALSE, "empty filename decoding FAILED");
|
||||||
|
+ return;
|
||||||
|
+ }
|
||||||
|
+ g_free (disposition);
|
||||||
|
+ g_assert_false (g_hash_table_contains (params, "filename"));
|
||||||
|
g_hash_table_destroy (params);
|
||||||
|
|
||||||
|
soup_message_headers_free (hdrs);
|
||||||
|
--
|
||||||
|
2.34.1
|
||||||
|
|
|
@ -12,7 +12,8 @@ DEPENDS = "glib-2.0 glib-2.0-native libxml2 sqlite3 libpsl"
|
||||||
SHRT_VER = "${@d.getVar('PV').split('.')[0]}.${@d.getVar('PV').split('.')[1]}"
|
SHRT_VER = "${@d.getVar('PV').split('.')[0]}.${@d.getVar('PV').split('.')[1]}"
|
||||||
|
|
||||||
SRC_URI = "${GNOME_MIRROR}/libsoup/${SHRT_VER}/libsoup-${PV}.tar.xz \
|
SRC_URI = "${GNOME_MIRROR}/libsoup/${SHRT_VER}/libsoup-${PV}.tar.xz \
|
||||||
file://0001-Fix-build-with-libxml2-2.12.0-and-clang-17.patch"
|
file://0001-Fix-build-with-libxml2-2.12.0-and-clang-17.patch \
|
||||||
|
file://0001-CVE-2025-32911.patch"
|
||||||
SRC_URI[sha256sum] = "e4b77c41cfc4c8c5a035fcdc320c7bc6cfb75ef7c5a034153df1413fa1d92f13"
|
SRC_URI[sha256sum] = "e4b77c41cfc4c8c5a035fcdc320c7bc6cfb75ef7c5a034153df1413fa1d92f13"
|
||||||
|
|
||||||
CVE_PRODUCT = "libsoup"
|
CVE_PRODUCT = "libsoup"
|
||||||
|
|
Loading…
Reference in New Issue
Block a user