stop the infinite loop of freeing the same connection_t over and

over when it's not linked into the connection_array


svn:r3474

1 files changed, 1 insertions, 1 deletions

diff --git a/src/or/main.c b/src/or/main.c
index d47f0c762..edcc0a8dc 100644
--- a/src/or/main.c
+++ b/src/or/main.c
@@ -189,8 +189,8 @@ static void connection_unlink(connection_t *conn, int remove) {
   connection_about_to_close_connection(conn);
   if (remove) {
     connection_remove(conn);
-    smartlist_remove(closeable_connection_lst, conn);
   }
+  smartlist_remove(closeable_connection_lst, conn);
   if (conn->type == CONN_TYPE_EXIT) {
     assert_connection_edge_not_dns_pending(conn);
   }