com php-src: Don't immediately destroy curl pushfunction: ext/curl/multi.c
| From: | Nikita Popov | Date: | Fri, 15 Mar 2019 16:07:19 +0000 |
| Subject: | com php-src: Don't immediately destroy curl pushfunction: ext/curl/multi.c | ||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-105155@lists.php.net to get a copy of this message | ||
Commit: 97f9fd6949a52d1eb30c3e1c90f0789b57c29468
Author: Nikita Popov <nikita.ppv@gmail.com> Fri, 15 Mar 2019 17:06:34 +0100
Parents: e7d40afb7a7984174eb132a14b7a6621c8e76258
Branches: PHP-7.2 PHP-7.3 PHP-7.4 master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=97f9fd6949a52d1eb30c3e1c90f0789b57c29468
Log:
Don't immediately destroy curl pushfunction
Changed paths:
M ext/curl/multi.c
Diff:
diff --git a/ext/curl/multi.c b/ext/curl/multi.c
index 073a6b3..0d55e75 100644
--- a/ext/curl/multi.c
+++ b/ext/curl/multi.c
@@ -425,6 +425,7 @@ void _php_curl_multi_close(zend_resource *rsrc) /* {{{ */
curl_multi_cleanup(mh->multi);
zend_llist_clean(&mh->easyh);
if (mh->handlers->server_push) {
+ zval_ptr_dtor(&mh->handlers->server_push->func_name);
efree(mh->handlers->server_push);
}
if (mh->handlers) {
@@ -587,11 +588,6 @@ static int _php_curl_multi_setopt(php_curlm *mh, zend_long option, zval
*zvalue,
ZVAL_COPY(&mh->handlers->server_push->func_name, zvalue);
mh->handlers->server_push->method = PHP_CURL_USER;
- if (!Z_ISUNDEF(mh->handlers->server_push->func_name)) {
- zval_ptr_dtor(&mh->handlers->server_push->func_name);
- mh->handlers->server_push->fci_cache = empty_fcall_info_cache;
-
- }
error = curl_multi_setopt(mh->multi, option, _php_server_push_callback);
if (error != CURLM_OK) {
return 0;