From: Nick Mathewson <nickm@torproject.org>
Date: Sat, 29 Dec 2012 03:38:42 +0000 (-0500)
Subject: Fix memory leak in safe-cookie authentication code
X-Git-Tag: tor-0.2.4.11-alpha~58^2~2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=4b571d3ab3d4c8e13fe0bc73f15431294d19615a;p=thirdparty%2Ftor.git

Fix memory leak in safe-cookie authentication code

Coverity spotted this. Bug 7816. Fix on 0.2.3.13-alpha.
---

diff --git a/changes/bug7816_023 b/changes/bug7816_023
new file mode 100644
index 0000000000..cfa754a996
--- /dev/null
+++ b/changes/bug7816_023
@@ -0,0 +1,3 @@
+  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.
diff --git a/src/or/control.c b/src/or/control.c
index 913d18a7fc..fc7bae23e1 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -3099,6 +3099,8 @@ handle_control_authchallenge(control_connection_t *conn, uint32_t len,
                            "SERVERNONCE=%s\r\n",
                            server_hash_encoded,
                            server_nonce_encoded);
+
+  tor_free(client_nonce);
   return 0;
 }