Never disable debugger attachment for the unit tests

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.
+

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 "