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.

1 files changed, 3 insertions, 4 deletions

diff --git a/lib/url.c b/lib/url.c
index 887c06a84..fe9c322bf 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -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);