diff options
author | Daniel Stenberg <daniel@haxx.se> | 2018-06-20 23:00:36 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2018-06-20 23:00:36 +0200 |
commit | 3475034031b480d553af11d43519d068e190472b (patch) | |
tree | 7ca074fe0c000a1527daa0e80237739536852697 | |
parent | dfb873e308eb19ee31064bb2a398f7da7b58873c (diff) | |
download | curl-bagder/dangling-easy-pointer.tar.gz |
url: use a fresh easy handle with connections from the cachebagder/dangling-easy-pointer
... and make sure to NULLify the ->data pointer when the connection is put
into the cache to make this mistake easier to detect in the future.
Reported-by: Will Dietz
Closes #2669
-rw-r--r-- | lib/conncache.c | 3 | ||||
-rw-r--r-- | lib/url.c | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/lib/conncache.c b/lib/conncache.c index 6bd06582a..066542915 100644 --- a/lib/conncache.c +++ b/lib/conncache.c @@ -6,7 +6,7 @@ * \___|\___/|_| \_\_____| * * Copyright (C) 2012 - 2016, Linus Nielsen Feltzing, <linus@haxx.se> - * Copyright (C) 2012 - 2017, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 2012 - 2018, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -451,6 +451,7 @@ bool Curl_conncache_return_conn(struct connectdata *conn) } CONN_LOCK(data); conn->inuse = FALSE; /* Mark the connection unused */ + conn->data = NULL; /* no owner */ CONN_UNLOCK(data); return (conn_candidate == conn) ? FALSE : TRUE; @@ -965,6 +965,7 @@ static bool extract_if_dead(struct connectdata *conn, use */ bool dead; + conn->data = data; if(conn->handler->connection_check) { /* The protocol has a special method for checking the state of the connection. Use it to check if the connection is dead. */ @@ -979,7 +980,6 @@ static bool extract_if_dead(struct connectdata *conn, } if(dead) { - conn->data = data; infof(data, "Connection %ld seems to be dead!\n", conn->connection_id); Curl_conncache_remove_conn(conn, FALSE); return TRUE; |