On Thu, Sep 8, 2011 at 21:53, Alexey Shein <con...@gmail.com> wrote: > Hello, Hannes! > It seems your fix is wrong, bug54798 still fails on my machine on > trunk, 5.4 and 5.3 branches (ubuntu x64). > IMO you should move your patch one line upper, i.e. before call to > _php_curl_cleanup_handle(ch); since error is rasing there.
Hmmh. Your right, it is failing. Maybe I screwed something up while merging between the branches. I'll check it out. > There's also related bug with curl_multi part, see bug48203_multi.phpt > These tests fail for me now on trunk (it seems I forgot to add some of > them to other branches, please, consider trunk first): It was quite confusing why some tests were passing in 5_4, but the same test didn't on trunk. Turns out the trunk test wasn't even remotely similar to the 5_4 test yesterday :) I've merged the test cases now so it should be easier to figure out what is going on :] There are several memleaks also detected by the test suite that needs to be looked at. -Hannes -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php