From: Daniel Stenberg <daniel@haxx.se>
Date: Thu, 25 Oct 2007 09:41:35 +0000 (+0000)
Subject: Don't assume there's a sessionhandle around when a connection is disconnected,
X-Git-Tag: curl-7_17_1~23
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ca67dcbc055bb1a017e9d215859362348dbbcbaa;p=curl

Don't assume there's a sessionhandle around when a connection is disconnected,
so do the data->reqdata.current_conn assignment when we know there is an easy
handle existing! Fixes the valgrind report on test 509.
---

diff --git a/lib/url.c b/lib/url.c
index 9d6be5315..c87eeaa81 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -2058,9 +2058,6 @@ static void conn_free(struct connectdata *conn)
   if(CURL_SOCKET_BAD != conn->sock[FIRSTSOCKET])
     sclose(conn->sock[FIRSTSOCKET]);
 
-  if (conn->data->reqdata.current_conn == conn) {
-    conn->data->reqdata.current_conn = NULL;
-  }
   Curl_safefree(conn->user);
   Curl_safefree(conn->passwd);
   Curl_safefree(conn->proxyuser);
@@ -2170,6 +2167,7 @@ CURLcode Curl_disconnect(struct connectdata *conn)
   }
 
   conn_free(conn);
+  data->reqdata.current_conn = NULL;
 
   return CURLE_OK;
 }