From a6c48c8be71da189ae64e33dae6ab50c023c9ca0 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Tue, 2 Sep 2014 23:44:42 +0200 Subject: curl_multi_cleanup: remove superfluous NULL assigns ... as the struct is free()d in the end anyway. It was first pointed out to me that one of the ->msglist assignments were supposed to have been ->pending but was a copy and paste mistake when I realized none of the clearing of pointers had to be there. --- lib/multi.c | 11 ----------- 1 file changed, 11 deletions(-) (limited to 'lib/multi.c') diff --git a/lib/multi.c b/lib/multi.c index 7947436c7..a680ce916 100644 --- a/lib/multi.c +++ b/lib/multi.c @@ -1881,22 +1881,12 @@ CURLMcode curl_multi_cleanup(CURLM *multi_handle) multi->closure_handle->dns.hostcache); Curl_close(multi->closure_handle); - multi->closure_handle = NULL; } Curl_hash_destroy(multi->sockhash); - multi->sockhash = NULL; - Curl_conncache_destroy(multi->conn_cache); - multi->conn_cache = NULL; - - /* remove the pending list of messages */ Curl_llist_destroy(multi->msglist, NULL); - multi->msglist = NULL; - - /* remove the pending handles queue */ Curl_llist_destroy(multi->pending, NULL); - multi->msglist = NULL; /* remove all easy handles */ data = multi->easyp; @@ -1917,7 +1907,6 @@ CURLMcode curl_multi_cleanup(CURLM *multi_handle) } Curl_hash_destroy(multi->hostcache); - multi->hostcache = NULL; /* Free the blacklists by setting them to NULL */ Curl_pipeline_set_site_blacklist(NULL, &multi->pipelining_site_bl); -- cgit v1.2.3