stop the infinite loop of freeing the same connection_t over and

over when it's not linked into the connection_array


svn:r3474

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);
   }