[tor-commits] [tor/master] Free pending_entry_connections on shutdown.

nickm at torproject.org nickm at torproject.org
Thu Nov 19 15:44:39 UTC 2015


commit 58edf92678019c0e69c043976b2af06551d83ed3
Author: Nick Mathewson <nickm at torproject.org>
Date:   Tue Nov 17 09:06:47 2015 -0500

    Free pending_entry_connections on shutdown.
---
 src/or/connection_edge.c |    9 ++++++++-
 src/or/connection_edge.h |    2 ++
 src/or/main.c            |    1 +
 3 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c
index 20a27dc..67e594a 100644
--- a/src/or/connection_edge.c
+++ b/src/or/connection_edge.c
@@ -509,7 +509,6 @@ connection_edge_finished_connecting(edge_connection_t *edge_conn)
  * (Right now, we check in several places to make sure that this list is
  * correct.  When it's incorrect, we'll fix it, and log a BUG message.)
  */
-/* XXXXX Free this list on exit. */
 static smartlist_t *pending_entry_connections = NULL;
 
 static int untried_pending_connections = 0;
@@ -3591,3 +3590,11 @@ circuit_clear_isolation(origin_circuit_t *circ)
   circ->socks_username_len = circ->socks_password_len = 0;
 }
 
+/** Free all storage held in module-scoped variables for connection_edge.c */
+void
+connection_edge_free_all(void)
+{
+  untried_pending_connections = 0;
+  smartlist_free(pending_entry_connections);
+  pending_entry_connections = NULL;
+}
diff --git a/src/or/connection_edge.h b/src/or/connection_edge.h
index 86f3fe9..521e759 100644
--- a/src/or/connection_edge.h
+++ b/src/or/connection_edge.h
@@ -102,6 +102,8 @@ int connection_edge_update_circuit_isolation(const entry_connection_t *conn,
 void circuit_clear_isolation(origin_circuit_t *circ);
 streamid_t get_unique_stream_id_by_circ(origin_circuit_t *circ);
 
+void connection_edge_free_all(void);
+
 /** @name Begin-cell flags
  *
  * These flags are used in RELAY_BEGIN cells to change the default behavior
diff --git a/src/or/main.c b/src/or/main.c
index 0fe818d..b5c1b8b 100644
--- a/src/or/main.c
+++ b/src/or/main.c
@@ -3084,6 +3084,7 @@ tor_free_all(int postfork)
   channel_tls_free_all();
   channel_free_all();
   connection_free_all();
+  connection_edge_free_all();
   scheduler_free_all();
   memarea_clear_freelist();
   nodelist_free_all();





More information about the tor-commits mailing list