[tor-commits] [tor/maint-0.4.2] Do not set "once" when calling tor_bug_occurred_ from BUG().

nickm at torproject.org nickm at torproject.org
Mon Jun 29 17:49:16 UTC 2020


commit 2a1f8ea2e78486966c71762ebc3286f2b3e531ff
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Jan 30 09:43:34 2020 -0500

    Do not set "once" when calling tor_bug_occurred_ from BUG().
    
    The "once" flag makes tor_bug_occurred_() say that future instances
    of the warning will be suppressed -- but that's not something that
    BUG() does.  Fixes bug 33095; bugfix on 0.4.1.1-alpha.
---
 changes/bug33095_041   | 5 +++++
 src/lib/log/util_bug.h | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/changes/bug33095_041 b/changes/bug33095_041
new file mode 100644
index 000000000..7d1f04e27
--- /dev/null
+++ b/changes/bug33095_041
@@ -0,0 +1,5 @@
+  o Minor bugfixes (logging, bug reporting):
+    - When logging a bug, do not say "Future instances of this warning
+      will be silenced" unless we are actually going to do
+      so. Previously we would say this whenever a BUG() check failed in
+      the code.  Fixes bug 33095; bugfix on 0.4.1.1-alpha.
diff --git a/src/lib/log/util_bug.h b/src/lib/log/util_bug.h
index 546ae1e3e..f67c1e995 100644
--- a/src/lib/log/util_bug.h
+++ b/src/lib/log/util_bug.h
@@ -196,7 +196,7 @@
   STMT_END
 #define BUG(cond)                                                       \
   (ASSERT_PREDICT_UNLIKELY_(cond) ?                                     \
-  (tor_bug_occurred_(SHORT_FILE__,__LINE__,__func__,"!("#cond")",1,NULL),1) \
+  (tor_bug_occurred_(SHORT_FILE__,__LINE__,__func__,"!("#cond")",0,NULL),1) \
    : 0)
 #endif /* defined(ALL_BUGS_ARE_FATAL) || ... */
 





More information about the tor-commits mailing list