[tor-commits] [tor/main] hs: Improve warning for bad service version

[dgoulet@xxxxxxxxxxxxxx]

commit db297a177edc8aba9334d6b3b525604d9b0f87a2
Author: David Goulet <dgoulet@xxxxxxxxxxxxxx>
Date:   Thu Oct 14 09:56:10 2021 -0400

    hs: Improve warning for bad service version
    
    Now that we don't have version 2, it gives us:
    
      [warn] HiddenServiceVersion must be between 3 and 3, not 2.
    
    This commit changes it to:
    
      [warn] HiddenServiceVersion must be 3, not 2.
    
    Part of #40476
    
    Signed-off-by: David Goulet <dgoulet@xxxxxxxxxxxxxx>
---
 src/feature/hs/hs_config.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/feature/hs/hs_config.c b/src/feature/hs/hs_config.c
index 7ffc7ecb96..f8d71674de 100644
--- a/src/feature/hs/hs_config.c
+++ b/src/feature/hs/hs_config.c
@@ -179,8 +179,12 @@ static bool
 check_value_oob(int i, const char *name, int low, int high)
 {
   if (i < low || i > high) {
-    log_warn(LD_CONFIG, "%s must be between %d and %d, not %d.",
-             name, low, high, i);
+    if (low == high) {
+      log_warn(LD_CONFIG, "%s must be %d, not %d.", name, low, i);
+    } else {
+      log_warn(LD_CONFIG, "%s must be between %d and %d, not %d.",
+               name, low, high, i);
+    }
     return true;
   }
   return false;



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits