[tor-commits] [tor/master] Avoid leaking headers received from SSL proxy

nickm at torproject.org nickm at torproject.org
Thu Feb 7 20:20:58 UTC 2013


commit b509ead20d93277697c1cb961a2ae520bce7fd7f
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Dec 28 22:44:45 2012 -0500

    Avoid leaking headers received from SSL proxy
    
    Fixes part of 7816. Spotted by coverity. Fix on 0.2.2.1-alpha.
---
 changes/bug7816_023 |    4 ++++
 src/or/connection.c |    1 +
 2 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/changes/bug7816_023 b/changes/bug7816_023
index cfa754a..a453029 100644
--- a/changes/bug7816_023
+++ b/changes/bug7816_023
@@ -1,3 +1,7 @@
   o Minor bugfixes (memory leak, controller):
     - Fix a memory leak during safe-cookie controller authentication.
       Spotted by Coverity. Fixes part of bug 7816; bugfix on 0.2.3.13-alpha.
+
+  o Minor bugfixes (memory leak, HTTPS proxy support):
+    - Fix a memory leak when receiving headers from an HTTPS proxy.
+      Spotted by Coverity. Fixes part of bug 7816; bugfix on 0.2.1.1-alpha.
diff --git a/src/or/connection.c b/src/or/connection.c
index eac9c4f..4c68262 100644
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -1607,6 +1607,7 @@ connection_read_https_proxy_response(connection_t *conn)
     tor_free(headers);
     return -1;
   }
+  tor_free(headers);
   if (!reason) reason = tor_strdup("[no reason given]");
 
   if (status_code == 200) {





More information about the tor-commits mailing list