aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorteor <teor@torproject.org>2020-01-16 10:00:36 +1000
committerteor <teor@torproject.org>2020-01-16 10:00:36 +1000
commit8e3e8b0a1c27a7643706259ff7413e4e2d359124 (patch)
tree6d1e13601de85bda52473fd51aa55bf019a4fb7c
parent262906e99430f41012f4d699d2ac3eecf3a8c1dd (diff)
parent026f068bb3aed9de0f1cc6792f6c9528b7b27db2 (diff)
downloadtor-8e3e8b0a1c27a7643706259ff7413e4e2d359124.tar.gz
tor-8e3e8b0a1c27a7643706259ff7413e4e2d359124.zip
Merge branch 'maint-0.4.0' into release-0.4.0
-rw-r--r--changes/ticket324074
-rw-r--r--src/core/mainloop/mainloop.c4
-rw-r--r--src/lib/evloop/compat_libevent.c10
-rw-r--r--src/lib/evloop/compat_libevent.h3
4 files changed, 21 insertions, 0 deletions
diff --git a/changes/ticket32407 b/changes/ticket32407
new file mode 100644
index 0000000000..badb09abfe
--- /dev/null
+++ b/changes/ticket32407
@@ -0,0 +1,4 @@
+ o Minor bugfixes (crash):
+ - When running Tor with an option like --verify-config or --dump-config
+ that does not start the event loop, avoid crashing if we try to exit
+ early because of an error. Fixes bug 32407; bugfix on 0.3.3.1-alpha.
diff --git a/src/core/mainloop/mainloop.c b/src/core/mainloop/mainloop.c
index e8de578b67..a42a8acc80 100644
--- a/src/core/mainloop/mainloop.c
+++ b/src/core/mainloop/mainloop.c
@@ -772,6 +772,10 @@ tor_shutdown_event_loop_and_exit(int exitcode)
main_loop_should_exit = 1;
main_loop_exit_value = exitcode;
+ if (! tor_libevent_is_initialized()) {
+ return; /* No event loop to shut down. */
+ }
+
/* Die with an assertion failure in ten seconds, if for some reason we don't
* exit normally. */
/* XXXX We should consider this code if it's never used. */
diff --git a/src/lib/evloop/compat_libevent.c b/src/lib/evloop/compat_libevent.c
index 91eacb9938..939d77f857 100644
--- a/src/lib/evloop/compat_libevent.c
+++ b/src/lib/evloop/compat_libevent.c
@@ -181,6 +181,16 @@ tor_libevent_initialize(tor_libevent_cfg *torcfg)
event_get_version(), tor_libevent_get_method());
}
+/**
+ * Return true iff the libevent module has been successfully initialized,
+ * and not subsequently shut down.
+ **/
+bool
+tor_libevent_is_initialized(void)
+{
+ return the_event_base != NULL;
+}
+
/** Return the current Libevent event base that we're set up to use. */
MOCK_IMPL(struct event_base *,
tor_libevent_get_base, (void))
diff --git a/src/lib/evloop/compat_libevent.h b/src/lib/evloop/compat_libevent.h
index afe887a013..92724c369c 100644
--- a/src/lib/evloop/compat_libevent.h
+++ b/src/lib/evloop/compat_libevent.h
@@ -13,6 +13,8 @@
#include "lib/testsupport/testsupport.h"
#include "lib/malloc/malloc.h"
+#include <stdbool.h>
+
void configure_libevent_logging(void);
void suppress_libevent_log_msg(const char *msg);
@@ -68,6 +70,7 @@ typedef struct tor_libevent_cfg {
} tor_libevent_cfg;
void tor_libevent_initialize(tor_libevent_cfg *cfg);
+bool tor_libevent_is_initialized(void);
MOCK_DECL(struct event_base *, tor_libevent_get_base, (void));
const char *tor_libevent_get_method(void);
void tor_check_libevent_header_compatibility(void);