Skip to content

Commit

Permalink
fix(tor): don't close controlling socket again
Browse files Browse the repository at this point in the history
  • Loading branch information
bassosimone committed Feb 2, 2023
1 parent 952d063 commit 06cf94e
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions CDEPS/tor/003.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
diff --git a/src/feature/api/tor_api.c b/src/feature/api/tor_api.c
index 88e91ebfd5..2773949264 100644
--- a/src/feature/api/tor_api.c
+++ b/src/feature/api/tor_api.c
@@ -131,9 +131,13 @@ tor_main_configuration_free(tor_main_configuration_t *cfg)
}
raw_free(cfg->argv_owned);
}
+ /* See https://github.com/ooni/probe/issues/2405 to understand
+ why we're not closing the controller socker here. */
+ /*
if (SOCKET_OK(cfg->owning_controller_socket)) {
raw_closesocket(cfg->owning_controller_socket);
}
+ */
raw_free(cfg);
}

0 comments on commit 06cf94e

Please sign in to comment.