MFB: Fixed bug #46711 (cURL curl_setopt leaks memory in foreach loops).

diff --git a/ext/curl/interface.c b/ext/curl/interface.c
index 0efb2384daabdc51a2253646634a8f841ec926fb..6967d2c4e20d37a0cd87fd5d13d876cf2f851127 100644 (file)
--- a/ext/curl/interface.c
+++ b/ext/curl/interface.c
@@ -1793,17 +1793,17 @@ type_conflict:
    Set an option for a cURL transfer */
 PHP_FUNCTION(curl_setopt)
 {
-       zval       *zid, *zvalue;
+       zval       *zid, **zvalue;
        long       options;
        php_curl    *ch;
 
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "zlz", &zid, &options, &zvalue) == FAILURE) {
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "zlZ", &zid, &options, &zvalue) == FAILURE) {
                RETURN_FALSE;
        }
 
        ZEND_FETCH_RESOURCE(ch, php_curl *, &zid, -1, le_curl_name, le_curl);
 
-       if (!_php_curl_setopt(ch, options, &zvalue, return_value TSRMLS_CC)) {
+       if (!_php_curl_setopt(ch, options, zvalue, return_value TSRMLS_CC)) {
                RETURN_TRUE;
        } else {
                RETURN_FALSE;