[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Fix a double-free in failing case of handle_control_authenticate.



commit 801f4d438460cd6eb0190fc18161ad499a95bc9b
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Thu Sep 18 11:13:57 2014 -0400

    Fix a double-free in failing case of handle_control_authenticate.
    
    Bugfix on ed8f020e205267e6270494634346ab68d830e1d8; bug not in any
    released version of Tor.  Found by Coverity; this is CID 1239290.
    
    [Yes, I used this commit message before, in 58e813d0fcfcecfc2017.
    Turns out, that fix wasn't right, since I didn't look up a
    screen. :P ]
---
 src/or/control.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/src/or/control.c b/src/or/control.c
index b1709e0..92dd230 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -1173,6 +1173,7 @@ handle_control_authenticate(control_connection_t *conn, uint32_t len,
       bad_password = 1;
       SMARTLIST_FOREACH(sl, char *, cp, tor_free(cp));
       smartlist_free(sl);
+      sl = NULL;
     } else {
       SMARTLIST_FOREACH(sl, char *, expected,
       {

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits