Optimize native curl (#4101)

* reset active_handles when select returns * defense
swoole · Mar 16, 2021 · 62c01b0 · 62c01b0
1 parent 1267b2e
commit 62c01b0
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/thirdparty/php/curl/curl_multi.h b/thirdparty/php/curl/curl_multi.h
@@ -103,6 +103,7 @@ class cURLMulti {
     void del_timer() {
         if (timer && swoole_timer_is_available()) {
             swoole_timer_del(timer);
+            timer = nullptr;
         }
     }
 
@@ -153,7 +154,9 @@ class cURLMulti {
         }
 
         if (selector->active_handles.size() > 0) {
-            return selector->active_handles.size();
+            auto count = selector->active_handles.size();
+            selector->active_handles.clear();
+            return count;
         }
 
         zval _return_value;
@@ -195,6 +198,7 @@ class cURLMulti {
                 [this](void *data) {
                     zval result;
                     ZVAL_LONG(&result, selector->active_handles.size());
+                    selector->active_handles.clear();
                     PHPCoroutine::resume_m(selector->context, &result);
                 },
                 nullptr);