From: Daniel Stenberg <daniel@haxx.se>
Date: Tue, 4 Mar 2025 13:00:03 +0000 (+0100)
Subject: multi: start the loop over when handles are removed
X-Git-Tag: curl-8_13_0~241
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a66ede53092dcca9c908444598e0ba07cce0ccb0;p=thirdparty%2Fcurl.git

multi: start the loop over when handles are removed

Since more than one handle can be removed in a single call to
multi_runsingle(), we cannot easily continue on the next node when a
node has been removed since that node migth ALSO have been removed.

Reported-by: Philippe Antoine
Closes #16588
---

diff --git a/lib/multi.c b/lib/multi.c
index ae221d4f46..f347580702 100644
--- a/lib/multi.c
+++ b/lib/multi.c
@@ -2601,6 +2601,7 @@ CURLMcode curl_multi_perform(CURLM *m, int *running_handles)
   for(e = Curl_llist_head(&multi->process); e; e = n) {
     struct Curl_easy *data = Curl_node_elem(e);
     CURLMcode result;
+    unsigned int num_alive = multi->num_alive;
     /* Do the loop and only alter the signal ignore state if the next handle
        has a different NO_SIGNAL state than the previous */
     if(first) {
@@ -2619,6 +2620,12 @@ CURLMcode curl_multi_perform(CURLM *m, int *running_handles)
       if(result)
         returncode = result;
     }
+    if(num_alive != multi->num_alive)
+      /* Since more than one handle can be removed in a single call to
+         multi_runsingle(), we cannot easily continue on the next node when a
+         node has been removed since that node might ALSO have been
+         removed. */
+      n = Curl_llist_head(&multi->process);
   }
 
   sigpipe_apply(multi->admin, &pipe_st);