Send a CRLF at the end of a STATUS_* event, not in the middle of it

Fixes bug 6094; bugfix on commit 3a9351b57e528b1d0bd2e72bcf78db7c91b2ff8f.

diff --git a/changes/bug6094 b/changes/bug6094
new file mode 100644 (file)
index 0000000..835d127
--- /dev/null
+++ b/changes/bug6094
@@ -0,0 +1,8 @@
+  o Major bugfixes:
+
+    - Send a CRLF at the end of each STATUS_* control-port event.  A
+      change introduced in 0.2.3.16-alpha caused Tor to emit the CRLF
+      in the middle of a STATUS_* event; among other things, this
+      caused Vidalia to freeze.  Fixes bug 6094; bugfix on
+      0.2.3.16-alpha.
+

diff --git a/src/or/control.c b/src/or/control.c
index 8aa4240f12a9f9996efa5d0c15cbc87fc8634be3..782468f6fe82d6eae569837a057f4aaeef7b22ad 100644 (file)
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -3957,14 +3957,14 @@ control_event_status(int type, int severity, const char *format, va_list args)
       log_warn(LD_BUG, "Unrecognized status severity %d", severity);
       return -1;
   }
-  if (tor_snprintf(format_buf, sizeof(format_buf), "650 %s %s\r\n",
+  if (tor_snprintf(format_buf, sizeof(format_buf), "650 %s %s",
                    status, sev)<0) {
     log_warn(LD_BUG, "Format string too long.");
     return -1;
   }
   tor_vasprintf(&user_buf, format, args);
 
-  send_control_event(type, ALL_FORMATS, "%s %s", format_buf, user_buf);
+  send_control_event(type, ALL_FORMATS, "%s %s\r\n", format_buf, user_buf);
   tor_free(user_buf);
   return 0;
 }