diff options
author | Yang Tse <yangsita@gmail.com> | 2010-03-05 03:15:19 +0000 |
---|---|---|
committer | Yang Tse <yangsita@gmail.com> | 2010-03-05 03:15:19 +0000 |
commit | f4551a967820ab2be76fd5d600fbce32cd290e62 (patch) | |
tree | 63539c815d1817e8562753bf6ca98b6d41dece14 | |
parent | 52f088ec1bec69e35efd2ac3bb5b95d9ce82bd46 (diff) |
Constantine Sapuntzakis detected and fixed a double free in builds done
with threaded resolver enabled (Windows default configuration) that would
get triggered when a curl handle is closed while doing DNS resolution.
-rw-r--r-- | CHANGES | 5 | ||||
-rw-r--r-- | RELEASE-NOTES | 1 | ||||
-rw-r--r-- | lib/url.c | 7 |
3 files changed, 9 insertions, 4 deletions
@@ -6,6 +6,11 @@ Changelog +Yang Tse (5 Mar 2010) +- Constantine Sapuntzakis detected and fixed a double free in builds done + with threaded resolver enabled (Windows default configuration) that would + get triggered when a curl handle is closed while doing DNS resolution. + Daniel Stenberg (2 Mar 2010) - [Daniel Johnson] I've been trying to build libcurl with clang on Darwin and ran into some issues with the GSSAPI tests in configure.ac. The tests first diff --git a/RELEASE-NOTES b/RELEASE-NOTES index b96171cc5..d59cba5ae 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -28,6 +28,7 @@ This release includes the following bugfixes: o CURLOPT_CERTINFO memory leak o sub-second timeouts improvements o configure fixes for GSSAPI + o threaded resolver double free when closing curl handle This release includes the following known bugs: @@ -2490,12 +2490,11 @@ static void conn_free(struct connectdata *conn) Curl_llist_destroy(conn->done_pipe, NULL); /* possible left-overs from the async name resolvers */ -#if defined(CURLRES_ASYNCH) - Curl_safefree(conn->async.hostname); - Curl_safefree(conn->async.os_specific); #if defined(CURLRES_THREADED) Curl_destroy_thread_data(&conn->async); -#endif +#elif defined(CURLRES_ASYNCH) + Curl_safefree(conn->async.hostname); + Curl_safefree(conn->async.os_specific); #endif Curl_free_ssl_config(&conn->ssl_config); |