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 <[email protected]> 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 -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
