From: Daniel Stenberg <daniel@haxx.se>
Date: Wed, 20 Aug 2014 14:37:01 +0000 (+0200)
Subject: Curl_disconnect: don't free the URL
X-Git-Tag: curl-7_38_0~75
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=30f2d0c0b321cb17c38a2b21a50016e7901b55b2;p=curl

Curl_disconnect: don't free the URL

The URL is not a property of the connection so it should not be freed in
the connection disconnect but in the Curl_close() that frees the easy
handle.

Bug: http://curl.haxx.se/mail/lib-2014-08/0148.html
Reported-by: Paras S
---

diff --git a/lib/url.c b/lib/url.c
index cf9325308..e43b19d75 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -424,6 +424,12 @@ CURLcode Curl_close(struct SessionHandle *data)
   Curl_safefree(data->state.scratch);
   Curl_ssl_free_certinfo(data);
 
+  /* Cleanup possible redirect junk */
+  if(data->req.newurl) {
+    free(data->req.newurl);
+    data->req.newurl = NULL;
+  }
+
   if(data->change.referer_alloc) {
     Curl_safefree(data->change.referer);
     data->change.referer_alloc = FALSE;
@@ -2642,12 +2648,6 @@ CURLcode Curl_disconnect(struct connectdata *conn, bool dead_connection)
   /* Cleanup NTLM connection-related data */
   Curl_http_ntlm_cleanup(conn);
 
-  /* Cleanup possible redirect junk */
-  if(data->req.newurl) {
-    free(data->req.newurl);
-    data->req.newurl = NULL;
-  }
-
   if(conn->handler->disconnect)
     /* This is set if protocol-specific cleanups should be made */
     conn->handler->disconnect(conn, dead_connection);