Downgrade the unexpected sendme cell warnings for 0.2.4

See discussion on #8093

diff --git a/changes/bug8093.part1 b/changes/bug8093.part1
new file mode 100644 (file)
index 0000000..2450794
--- /dev/null
+++ b/changes/bug8093.part1
@@ -0,0 +1,3 @@
+  o Minor features:
+    - Downgrade "unexpected SENDME" warnings to protocol-warn for 0.2.4,
+      for bug 8093.

diff --git a/src/or/relay.c b/src/or/relay.c
index 52ff32f0ca9898e7c45bb870ba0ad360e8f43245..d57ceaacf89242599c51b1190edc11faceabb912 100644 (file)
--- a/src/or/relay.c
+++ b/src/or/relay.c
@@ -1494,9 +1494,8 @@ connection_edge_process_relay_cell(cell_t *cell, circuit_t *circ,
         if (layer_hint) {
           if (layer_hint->package_window + CIRCWINDOW_INCREMENT >
                 CIRCWINDOW_START_MAX) {
-            /*XXXX024: Downgrade this back to LOG_PROTOCOL_WARN after a while*/
-            log_fn(LOG_WARN, LD_PROTOCOL,
-                   "Bug/attack: unexpected sendme cell from exit relay. "
+            log_fn(LOG_PROTOCOL_WARN, LD_PROTOCOL,
+                   "Unexpected sendme cell from exit relay. "
                    "Closing circ.");
             return -END_CIRC_REASON_TORPROTOCOL;
           }
@@ -1507,9 +1506,8 @@ connection_edge_process_relay_cell(cell_t *cell, circuit_t *circ,
         } else {
           if (circ->package_window + CIRCWINDOW_INCREMENT >
                 CIRCWINDOW_START_MAX) {
-            /*XXXX024: Downgrade this back to LOG_PROTOCOL_WARN after a while*/
-            log_fn(LOG_WARN, LD_PROTOCOL,
-                   "Bug/attack: unexpected sendme cell from client. "
+            log_fn(LOG_PROTOCOL_WARN, LD_PROTOCOL,
+                   "Unexpected sendme cell from client. "
                    "Closing circ (window %d).",
                    circ->package_window);
             return -END_CIRC_REASON_TORPROTOCOL;