Re: [PHP-CVS] svn: php/php-src/ branches/PHP_5_2/NEWS branches/PHP_5_2/ext/curl/interface.c branches/PHP_5_2/ext/curl/php_curl.h branches/PHP_5_2/ext/curl/tests/curl_copy_handle_basic_007.phpt branches/PHP_5_3/NEWS branches/PHP_5_3/ext/curl/interface.c branches/PHP_5_3/ext/curl/php_curl.h branches/PHP_5_3/ext/curl/tests/curl_copy_handle_basic_007.phpt trunk/NEWS trunk/ext/curl/interface.c trunk/ext/curl/php_curl.h trunk/ext/curl/tests/curl_copy_handle_basic_007.phpt

Tue, 21 Jul 2009 16:00:30 -0700 

Sriram Natarajan wrote:

srinatar                                 Tue, 21 Jul 2009 20:32:32 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=284557

Log:
- Fixed bug #48774 (SIGSEGVs when using curl_copy_handle()).

Bug: http://bugs.php.net/48774 (Assigned) SIGSEGVs when using curl_copy_handle()
Changed paths: 
    U   php/php-src/branches/PHP_5_2/NEWS
    U   php/php-src/branches/PHP_5_2/ext/curl/interface.c
    U   php/php-src/branches/PHP_5_2/ext/curl/php_curl.h
    A   
php/php-src/branches/PHP_5_2/ext/curl/tests/curl_copy_handle_basic_007.phpt
    U   php/php-src/branches/PHP_5_3/NEWS
    U   php/php-src/branches/PHP_5_3/ext/curl/interface.c
    U   php/php-src/branches/PHP_5_3/ext/curl/php_curl.h
    A   
php/php-src/branches/PHP_5_3/ext/curl/tests/curl_copy_handle_basic_007.phpt
    U   php/php-src/trunk/NEWS
    U   php/php-src/trunk/ext/curl/interface.c
    U   php/php-src/trunk/ext/curl/php_curl.h
    A   php/php-src/trunk/ext/curl/tests/curl_copy_handle_basic_007.phpt
Just that you know: NEVER EVER commit before you have actually tested that the thing compiles at least. And preferrably passes the test(s).
Your patch for PHP_5_2 branch didn't quite apply like it should have. I fixed that now. I guess you can close the bug report now too? 

--Jani



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

Reply via email to