Never disable debugger attachment for the unit tests

diff --git a/changes/bug5448 b/changes/bug5448
new file mode 100644 (file)
index 0000000..aaf7dc7
--- /dev/null
+++ b/changes/bug5448
@@ -0,0 +1,5 @@
+  o Major bugfixes:
+    - Allow running the unit tests in gdb again. This was accidentally made
+      impossible when the DisableDebuggerAttachment option was introduced.
+      Fixes bug 5448; bugfix on 0.2.3.9-alpha.
+

diff --git a/src/or/config.c b/src/or/config.c
index 0c699b021d256543b7881e77e46494d80f3fb626..3de8426460190f10ef8f14e6d89c9966f073d2e0 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -1338,7 +1338,9 @@ options_act(const or_options_t *old_options)
     /* Remember if we already warned about being configured not to disable
      * debugger attachment */
     static int warned_debugger_attach = 0;
-    if (options->DisableDebuggerAttachment && !disabled_debugger_attach) {
+    /* Don't disable debugger attachment when we're running the unit tests. */
+    if (options->DisableDebuggerAttachment && !disabled_debugger_attach &&
+        running_tor) {
       int ok = tor_disable_debugger_attach();
       if (warned_debugger_attach && ok == 1) {
         log_notice(LD_CONFIG, "Disabled attaching debuggers for unprivileged "