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 <bj...@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

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to