public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Jan Macku <jamacku@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/curl] f44: Resolves: CVE-2026-9080 - UAF after pause in socket callback
Date: Mon, 24 Aug 2026 14:03:17 GMT [thread overview]
Message-ID: <178758019761.1.14445830222440048380.rpms-curl-2f242f26cf3e@fedoraproject.org> (raw)
A new commit has been pushed.
Repo : rpms/curl
Branch : f44
Commit : 2f242f26cf3eca5f27ec77c38908a5bafab03cd6
Author : Jan Macku <jamacku@redhat.com>
Date : 2026-08-24T14:46:04+02:00
Stats : +103/-0 in 2 file(s)
URL : https://src.fedoraproject.org/rpms/curl/c/2f242f26cf3eca5f27ec77c38908a5bafab03cd6?branch=f44
Log:
Resolves: CVE-2026-9080 - UAF after pause in socket callback
---
diff --git a/0021-curl-8.18.0-CVE-2026-9080.patch b/0021-curl-8.18.0-CVE-2026-9080.patch
new file mode 100644
index 0000000..08d1286
--- /dev/null
+++ b/0021-curl-8.18.0-CVE-2026-9080.patch
@@ -0,0 +1,96 @@
+From b443308bfaebea79d7355f2edf50df2a903c3dc8 Mon Sep 17 00:00:00 2001
+From: Daniel Stenberg <daniel@haxx.se>
+Date: Tue, 26 May 2026 09:52:19 +0200
+Subject: [PATCH] multi: handle pause in multi socket callback
+
+The mev_sh_entry object might be removed if curl_easy_pause() is called
+from within the socket callback.
+
+Introduced a 'magic' struct field to to 'mev_sh_entry' to make it easier
+to programmatically detect/assert if the pointer is bad - in debug
+builds.
+
+Reported-by: Joshua Rogers
+Closes #21748
+
+(cherry picked from commit 5ab34cba42e4ee4282fe8bab43f311d51b9bf9bd)
+---
+ lib/multi_ev.c | 23 ++++++++++++++++++++---
+ 1 file changed, 20 insertions(+), 3 deletions(-)
+
+diff --git a/lib/multi_ev.c b/lib/multi_ev.c
+index 0c89721f35..e0566e9ff5 100644
+--- a/lib/multi_ev.c
++++ b/lib/multi_ev.c
+@@ -40,6 +40,8 @@ static void mev_in_callback(struct Curl_multi *multi, bool value)
+ multi->in_callback = value;
+ }
+
++#define SH_ENTRY_MAGIC 0x570091d
++
+ /* Information about a socket for which we inform the libcurl application
+ * what to supervise (CURL_POLL_IN/CURL_POLL_OUT/CURL_POLL_REMOVE)
+ */
+@@ -51,6 +53,9 @@ struct mev_sh_entry {
+ * libcurl application to watch out for */
+ unsigned int readers; /* this many transfers want to read */
+ unsigned int writers; /* this many transfers want to write */
++#ifdef DEBUGBUILD
++ unsigned int magic;
++#endif
+ BIT(announced); /* this socket has been passed to the socket
+ callback at least once */
+ };
+@@ -75,6 +80,9 @@ static void mev_sh_entry_dtor(void *freethis)
+ {
+ struct mev_sh_entry *entry = (struct mev_sh_entry *)freethis;
+ Curl_uint32_spbset_destroy(&entry->xfers);
++#ifdef DEBUGBUILD
++ entry->magic = 0;
++#endif
+ curlx_free(entry);
+ }
+
+@@ -113,7 +121,9 @@ static struct mev_sh_entry *mev_sh_entry_add(struct Curl_hash *sh,
+ mev_sh_entry_dtor(check);
+ return NULL; /* major failure */
+ }
+-
++#ifdef DEBUGBUILD
++ check->magic = SH_ENTRY_MAGIC;
++#endif
+ return check; /* things are good in sockhash land */
+ }
+
+@@ -222,6 +232,7 @@ static CURLMcode mev_sh_entry_update(struct Curl_multi *multi,
+
+ /* we should only be called when the callback exists */
+ DEBUGASSERT(multi->socket_cb);
++ DEBUGASSERT(entry->magic == SH_ENTRY_MAGIC);
+ if(!multi->socket_cb)
+ return CURLM_OK;
+
+@@ -271,12 +282,18 @@ static CURLMcode mev_sh_entry_update(struct Curl_multi *multi,
+ rc = multi->socket_cb(data, s, comboaction, multi->socket_userp,
+ entry->user_data);
+ mev_in_callback(multi, FALSE);
+- entry->announced = TRUE;
+ if(rc == -1) {
+ multi->dead = TRUE;
+ return CURLM_ABORTED_BY_CALLBACK;
+ }
+- entry->action = (unsigned int)comboaction;
++ /* curl_easy_pause() is documented as callable from any callback; it
++ * re-enters mev_assess() which may free this 'entry'. Re-fetch. */
++ entry = mev_sh_entry_get(&multi->ev.sh_entries, s);
++ if(entry) {
++ DEBUGASSERT(entry->magic == SH_ENTRY_MAGIC);
++ entry->announced = TRUE;
++ entry->action = (unsigned int)comboaction;
++ }
+ return CURLM_OK;
+ }
+
+--
+2.55.0
+
diff --git a/curl.spec b/curl.spec
index 8ecf03b..2dff466 100644
--- a/curl.spec
+++ b/curl.spec
@@ -84,6 +84,9 @@ Patch019: 0019-curl-8.18.0-CVE-2026-9546.patch
# Fix exposing HTTP/3 early data (CVE-2026-9545)
Patch020: 0020-curl-8.18.0-CVE-2026-9545.patch
+# Fix UAF after pause in socket callback (CVE-2026-9080)
+Patch021: 0021-curl-8.18.0-CVE-2026-9080.patch
+
# patch making libcurl multilib ready
Patch101: 0101-curl-7.32.0-multilib.patch
@@ -305,6 +308,9 @@ be installed.
# <https://github.com/bagder/curl/commit/21e82bd6#commitcomment-12226582>
printf "1801\n" >>tests/data/DISABLED
+# disable test 1701 -- nghttpx rejects h2c upgrade with 400 Bad Request
+printf "1701\n" >>tests/data/DISABLED
+
# test3026: avoid pthread_create() failure due to resource exhaustion on i386
%ifarch %{ix86}
sed -e 's|NUM_THREADS 1000$|NUM_THREADS 256|' \
@@ -516,6 +522,7 @@ rm -f ${RPM_BUILD_ROOT}%{_mandir}/man1/wcurl.1*
- Fix env-set cross-proxy Digest auth state leak (CVE-2026-8927)
- Fix sending old referer (CVE-2026-9546)
- Fix exposing HTTP/3 early data (CVE-2026-9545)
+- Fix UAF after pause in socket callback (CVE-2026-9080)
* Wed Jul 29 2026 Jan Macku <jamacku@redhat.com> - 8.18.0-8
- Fix trailing dot domain super cookie (CVE-2026-8924)
reply other threads:[~2026-08-24 14:03 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=178758019761.1.14445830222440048380.rpms-curl-2f242f26cf3e@fedoraproject.org \
--to=jamacku@redhat.com \
--cc=git-commits@fedoraproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox