]> git.ipfire.org Git - thirdparty/curl.git/commit
multi: fix bad splay management
authorDaniel Stenberg <daniel@haxx.se>
Wed, 6 Aug 2025 07:03:31 +0000 (09:03 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Wed, 6 Aug 2025 12:13:36 +0000 (14:13 +0200)
commit019991c25ef0b0867381636f7f3d83b752e396be
treea10d7e5d8a68c86cf1a1dd10d2d352cdf6973dd6
parent6cebd35b4cda026d6f4ad682a51561be10b7e915
multi: fix bad splay management

The splay tree is a tree where each easy handle can be added *once*. The
expire time for that node is the closest expire time for that easy
handle.

Easy handles can however have more expire times queued up, so when the
node is removed from the splay tree because it is the next in line to
take care of, we must check if there is another expire time in the queue
and then add the node back into the splay.

Failing to do the later part, the calling of add_next_timeout after
Curl_splaygetbest, would leave the state.expiretime on the previous time
stamp, which when could make the next call to Curl_splaygetbest use the
wrong time stamp and get a wrong node out, causing trouble.

Reported-by: letshack9707 on hackerone
Closes #18201
lib/multi.c