From 1941fb2bf10adde43a49d929447589b4b6b4feab Mon Sep 17 00:00:00 2001 From: Jani Taskinen Date: Sat, 25 Jul 2009 13:00:25 +0000 Subject: [PATCH] - Fixed bug #49052 (context option headers freed too early when using --with-curlwrappers) --- NEWS | 2 ++ ext/curl/streams.c | 16 ++++++++++++---- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/NEWS b/NEWS index af3b0cfc9b..64a2f34f4f 100644 --- a/NEWS +++ b/NEWS @@ -4,6 +4,8 @@ PHP NEWS - Fixed regression in cURL extension that prevented flush of data to output defined as a file handle. (Ilia) +- Fixed bug #49052 (context option headers freed too early when using + --with-curlwrappers). (Jani) - Fixed bug #49032 (SplFileObject::fscanf() variables passed by reference). (Jani) - Fixed bug #48980 (Crash when compiling with pdo_firebird). (Felipe) diff --git a/ext/curl/streams.c b/ext/curl/streams.c index e56a3aec31..22ea72b0f3 100644 --- a/ext/curl/streams.c +++ b/ext/curl/streams.c @@ -469,8 +469,7 @@ php_stream *php_curl_stream_opener(php_stream_wrapper *wrapper, char *filename, #else php_error_docref(NULL TSRMLS_CC, E_WARNING, "There was an error mcode=%d", m); #endif - php_stream_close(stream); - return NULL; + goto exit_fail; } /* we have only one curl handle here, even though we use multi syntax, @@ -488,14 +487,23 @@ php_stream *php_curl_stream_opener(php_stream_wrapper *wrapper, char *filename, } } if (msg_found) { - php_stream_close(stream); - return NULL; + goto exit_fail; } } + + /* context headers are not needed anymore */ if (slist) { + curl_easy_setopt(curlstream->curl, CURLOPT_HTTPHEADER, NULL); curl_slist_free_all(slist); } return stream; + +exit_fail: + php_stream_close(stream); + if (slist) { + curl_slist_free_all(slist); + } + return NULL; } static php_stream_wrapper_ops php_curl_wrapper_ops = { -- 2.40.0