- Fixed bug #49052 (context option headers freed too early when using --with-curlwrappers)

This commit is contained in:
Jani Taskinen 2009-07-25 13:00:25 +00:00
parent ee992d9ff7
commit 4f760d11f8
2 changed files with 14 additions and 4 deletions

2
NEWS
View File

@ -7,6 +7,8 @@ PHP NEWS
Stas)
- Fixed signature generation/validation for zip archives in ext/phar. (Greg)
- 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 #49012 (phar tar signature algorithm reports as Unknown (0) in

View File

@ -474,8 +474,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,
@ -493,14 +492,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 = {