[tor-commits] [tor/maint-0.4.5] sendme: Turn log warning into debug

asn at torproject.org asn at torproject.org
Wed Nov 18 10:38:59 UTC 2020


commit 16351d655d5f2cf8a69323b69f55f89c96d91beb
Author: David Goulet <dgoulet at torproject.org>
Date:   Tue Nov 17 13:23:29 2020 -0500

    sendme: Turn log warning into debug
    
    When sending the stream level SENDME, it is possible the cirucit was marked
    for close or any other failures that can occur. These events can occur
    naturally.
    
    Fixes #40142
    
    Signed-off-by: David Goulet <dgoulet at torproject.org>
---
 changes/ticket40142  | 3 +++
 src/core/or/sendme.c | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/changes/ticket40142 b/changes/ticket40142
new file mode 100644
index 0000000000..25a96b3df3
--- /dev/null
+++ b/changes/ticket40142
@@ -0,0 +1,3 @@
+  o Minor bugfixes (logging, flow control):
+    - Turn a SENDME failure log warning into a debug. It can actually happen
+      naturally. Fixes bug 40142; bugfix on 0.4.1.1-alpha.
diff --git a/src/core/or/sendme.c b/src/core/or/sendme.c
index 7b4595c1ac..9cad245b29 100644
--- a/src/core/or/sendme.c
+++ b/src/core/or/sendme.c
@@ -398,8 +398,8 @@ sendme_connection_edge_consider_sending(edge_connection_t *conn)
     conn->deliver_window += STREAMWINDOW_INCREMENT;
     if (connection_edge_send_command(conn, RELAY_COMMAND_SENDME,
                                      NULL, 0) < 0) {
-      log_warn(LD_BUG, "connection_edge_send_command failed while sending "
-                       "a SENDME. Circuit probably closed, skipping.");
+      log_debug(LD_CIRC, "connection_edge_send_command failed while sending "
+                         "a SENDME. Circuit probably closed, skipping.");
       goto end; /* The circuit's closed, don't continue */
     }
   }



More information about the tor-commits mailing list