]> granicus.if.org Git - php/commitdiff
Fixed bug #52202 (CURLOPT_PRIVATE gets corrupted).
authorIlia Alshanetsky <iliaa@php.net>
Tue, 30 Nov 2010 13:40:02 +0000 (13:40 +0000)
committerIlia Alshanetsky <iliaa@php.net>
Tue, 30 Nov 2010 13:40:02 +0000 (13:40 +0000)
ext/curl/interface.c
ext/curl/tests/bug52202.phpt [new file with mode: 0644]

index 78c9821b6510f9f38bd9b54240a3468545afaaef..ad163d7cd0d69a5d8e2dc42ecd68d9154177a2e5 100644 (file)
@@ -1255,7 +1255,6 @@ static size_t curl_passwd(void *ctx, char *prompt, char *buf, int buflen)
 /* }}} */
 #endif
 
-#if LIBCURL_VERSION_NUM < 0x071101
 /* {{{ curl_free_string
  */
 static void curl_free_string(void **string)
@@ -1263,7 +1262,6 @@ static void curl_free_string(void **string)
        efree(*string);
 }
 /* }}} */
-#endif
 
 /* {{{ curl_free_post
  */
@@ -1340,9 +1338,7 @@ static void alloc_curl_handle(php_curl **ch)
 
        memset(&(*ch)->err, 0, sizeof((*ch)->err));
 
-#if LIBCURL_VERSION_NUM < 0x071101
        zend_llist_init(&(*ch)->to_free.str,   sizeof(char *),            (llist_dtor_func_t) curl_free_string, 0);
-#endif
        zend_llist_init(&(*ch)->to_free.slist, sizeof(struct curl_slist), (llist_dtor_func_t) curl_free_slist,  0);
        zend_llist_init(&(*ch)->to_free.post,  sizeof(struct HttpPost),   (llist_dtor_func_t) curl_free_post,   0);
 }
@@ -1559,11 +1555,10 @@ PHP_FUNCTION(curl_copy_handle)
        curl_easy_setopt(dupch->cp, CURLOPT_INFILE,            (void *) dupch);
        curl_easy_setopt(dupch->cp, CURLOPT_WRITEHEADER,       (void *) dupch);
 
-#if LIBCURL_VERSION_NUM < 0x071101
        zend_llist_copy(&dupch->to_free.str, &ch->to_free.str);
        /* Don't try to free copied strings, they're free'd when the original handle is destroyed */
        dupch->to_free.str.dtor = NULL;
-#endif
+
        zend_llist_copy(&dupch->to_free.slist, &ch->to_free.slist);
        zend_llist_copy(&dupch->to_free.post, &ch->to_free.post);
 
@@ -1752,14 +1747,22 @@ static int _php_curl_setopt(php_curl *ch, long option, zval **zvalue, zval *retu
                                        return 1;
                                }
                        } else {
+                               if (option == CURLOPT_PRIVATE) {
+                                       char *copystr;
+#if LIBCURL_VERSION_NUM < 0x071100
+string_copy:
+#endif
+                                       copystr = estrndup(Z_STRVAL_PP(zvalue), Z_STRLEN_PP(zvalue));
+                                       error = curl_easy_setopt(ch->cp, option, copystr);
+                                       zend_llist_add_element(&ch->to_free.str, &copystr);
+                               } else {
 #if LIBCURL_VERSION_NUM >= 0x071100
-                               /* Strings passed to libcurl as ’char *’ arguments, are copied by the library... NOTE: before 7.17.0 strings were not copied. */
-                               error = curl_easy_setopt(ch->cp, option, Z_STRVAL_PP(zvalue));
-#else
-                               copystr = estrndup(Z_STRVAL_PP(zvalue), Z_STRLEN_PP(zvalue));
-                               error = curl_easy_setopt(ch->cp, option, copystr);
-                               zend_llist_add_element(&ch->to_free.str, &copystr);
+                                       /* Strings passed to libcurl as ’char *’ arguments, are copied by the library... NOTE: before 7.17.0 strings were not copied. */
+                                       error = curl_easy_setopt(ch->cp, option, Z_STRVAL_PP(zvalue));
+#else                          
+                                       goto string_copy;                       
 #endif
+                               }
                        }
                        break;
                }
@@ -2466,9 +2469,7 @@ static void _php_curl_close_ex(php_curl *ch TSRMLS_DC)
        }
 
        curl_easy_cleanup(ch->cp);
-#if LIBCURL_VERSION_NUM < 0x071101
        zend_llist_clean(&ch->to_free.str);
-#endif
 
        /* cURL destructors should be invoked only by last curl handle */
        if (Z_REFCOUNT_P(ch->clone) <= 1) {
diff --git a/ext/curl/tests/bug52202.phpt b/ext/curl/tests/bug52202.phpt
new file mode 100644 (file)
index 0000000..a304d7f
--- /dev/null
@@ -0,0 +1,16 @@
+--TEST--
+Bug #52202 (CURLOPT_PRIVATE gets clobbered)
+--SKIPIF--
+<?php
+if (!extension_loaded('curl')) exit("skip curl extension not loaded");
+?>
+--FILE--
+<?php
+$curl = curl_init("http://www.google.com");
+curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
+curl_setopt($curl, CURLOPT_PRIVATE, "123");
+curl_exec($curl);
+
+var_dump(curl_getinfo($curl, CURLINFO_PRIVATE));
+--EXPECT--
+string(3) "123"