From 2578ec14f54c080a353b803ed5bbc84b90d91a3b Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 13 Mar 2014 10:46:59 -0400 Subject: [PATCH] Avoid double-close paths in http tests Coverity spotted that there were some paths through the test functions that would double-close some sockets. --- test/regress_http.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/regress_http.c b/test/regress_http.c index 98291500..0de22565 100644 --- a/test/regress_http.c +++ b/test/regress_http.c @@ -734,6 +734,7 @@ http_delete_test(void *arg) bufferevent_free(bev); evutil_closesocket(fd); + fd = -1; evhttp_free(http); @@ -2747,6 +2748,7 @@ http_incomplete_test_(struct basic_test_data *data, int use_timeout) bufferevent_free(bev); if (use_timeout) { evutil_closesocket(fd); + fd = -1; } evhttp_free(http); -- 2.50.1