[tor-commits] [tor/maint-0.4.1] More LOG_PROTOCOL_WARN.

nickm at torproject.org nickm at torproject.org
Wed Jun 26 14:09:02 UTC 2019


commit e54ce03b4f439d7cdf9d0d0da3be11b72a1a83fd
Author: Mike Perry <mikeperry-git at torproject.org>
Date:   Wed Jun 5 12:33:39 2019 -0700

    More LOG_PROTOCOL_WARN.
    
    Make origin-side messages about padding negotiation failure into
    LOG_PROTOCOL_WARN.
    
    I'm not sure I like this either.. But the negotiation refusal case might
    happen naturally due to consensus drift, and is functionally no different than
    a corrupted cell.
---
 src/core/or/circuitpadding.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/core/or/circuitpadding.c b/src/core/or/circuitpadding.c
index 84650a5a9..ebad6518b 100644
--- a/src/core/or/circuitpadding.c
+++ b/src/core/or/circuitpadding.c
@@ -2805,7 +2805,7 @@ circpad_handle_padding_negotiate(circuit_t *circ, cell_t *cell)
   circpad_negotiate_t *negotiate;
 
   if (CIRCUIT_IS_ORIGIN(circ)) {
-    log_fn(LOG_WARN, LD_CIRC,
+    log_fn(LOG_PROTOCOL_WARN, LD_CIRC,
            "Padding negotiate cell unsupported at origin.");
     return -1;
   }
@@ -2871,7 +2871,7 @@ circpad_handle_padding_negotiated(circuit_t *circ, cell_t *cell,
 
   /* Verify this came from the expected hop */
   if (!circpad_padding_is_from_expected_hop(circ, layer_hint)) {
-    log_fn(LOG_WARN, LD_CIRC,
+    log_fn(LOG_PROTOCOL_WARN, LD_CIRC,
            "Padding negotiated cell from wrong hop!");
     return -1;
   }
@@ -2898,7 +2898,7 @@ circpad_handle_padding_negotiated(circuit_t *circ, cell_t *cell,
     // and be sad
     free_circ_machineinfos_with_machine_num(circ, negotiated->machine_type);
     TO_ORIGIN_CIRCUIT(circ)->padding_negotiation_failed = 1;
-    log_fn(LOG_INFO, LD_CIRC,
+    log_fn(LOG_PROTOCOL_WARN, LD_CIRC,
            "Middle node did not accept our padding request.");
   }
 





More information about the tor-commits mailing list