Re: svn: /php/php-src/ branches/PHP_5_3/ext/curl/interface.c branches/PHP_5_3/ext/curl/multi.c branches/PHP_5_3/ext/curl/php_curl.h
branches/PHP_5_3/ext/curl/tests/bug48203_multi.phpt branches/PHP_5_3/ext/curl/tests/bug54798.phpt branches/P

hi, This fix seems incomplete or not fully tested (new leaks). Please revert it (feel free to keep it in trunk if you wish to) but revert it from 5.3/5.4 as both branches may have releases very soon and we do need clean tests before the release process begins. I will do it by tonight if you don't have the time. Cheers, On Mon, Sep 12, 2011 at 11:16 AM, Hannes Magnusson <bjori@php.net> wrote: > bjori                                    Mon, 12 Sep 2011 09:16:04 +0000 > > Revision: > http://svn.php.net/viewvc?view=revision&revision=316511 > > Log: > Fixed issues when streams were closed before curl read/write from them, or cleaning > Closing a original handle after copying it now no longer cleans up all resources (fixes missing > CURLOPT_POSTFIELDS values among others) > > Changed paths: >    U   php/php-src/branches/PHP_5_3/ext/curl/interface.c >    U   php/php-src/branches/PHP_5_3/ext/curl/multi.c >    U   php/php-src/branches/PHP_5_3/ext/curl/php_curl.h >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/bug48203_multi.phpt >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/bug54798.phpt >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/curl_copy_handle_basic_002.phpt >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/curl_copy_handle_basic_005.phpt >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/curl_file_deleted_before_curl_close.phpt >    U   php/php-src/branches/PHP_5_3/ext/curl/tests/curl_setopt_basic002.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/interface.c >    U   php/php-src/branches/PHP_5_4/ext/curl/multi.c >    U   php/php-src/branches/PHP_5_4/ext/curl/php_curl.h >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/bug48203_multi.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/bug54798.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/curl_copy_handle_basic_002.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/curl_copy_handle_basic_005.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/curl_file_deleted_before_curl_close.phpt >    U   php/php-src/branches/PHP_5_4/ext/curl/tests/curl_setopt_basic002.phpt >    U   php/php-src/trunk/ext/curl/interface.c >    U   php/php-src/trunk/ext/curl/multi.c >    U   php/php-src/trunk/ext/curl/php_curl.h >    U   php/php-src/trunk/ext/curl/tests/bug48203_multi.phpt >    U   php/php-src/trunk/ext/curl/tests/bug54798.phpt >    U   php/php-src/trunk/ext/curl/tests/curl_copy_handle_basic_002.phpt >    U   php/php-src/trunk/ext/curl/tests/curl_copy_handle_basic_005.phpt >    U   php/php-src/trunk/ext/curl/tests/curl_file_deleted_before_curl_close.phpt >    U   php/php-src/trunk/ext/curl/tests/curl_setopt_basic002.phpt > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org