Skip to content

Commit 4517e2f

Browse files
committed
Merge bitcoin/bitcoin#28147: suppressions: note that type:ClassName::MethodName should be used
d0c6cc4 suppressions: note that 'type:ClassName::MethodName' should be used (fanquake) Pull request description: Now that the symbolizer is back in play, suppressions can once-again be targeted to functions, rather than file-wide. ACKs for top commit: MarcoFalke: lgtm ACK d0c6cc4 hebasto: ACK d0c6cc4 Tree-SHA512: fb65398eae18a6ebc5f8414275c568cf2664ab5357c2b3160f3bf285b67bc3af788225c5dba3c824c0e098627789450bec775375f52529d71c6ef700a9632d65
2 parents 8fba5df + d0c6cc4 commit 4517e2f

File tree

1 file changed

+3
-6
lines changed
  • test/sanitizer_suppressions

1 file changed

+3
-6
lines changed

test/sanitizer_suppressions/ubsan

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
1+
# Suppressions should use `sanitize-type:ClassName::MethodName`.
2+
13
# -fsanitize=undefined suppressions
24
# =================================
3-
# The suppressions would be `sanitize-type:ClassName::MethodName`,
4-
# however due to a bug in clang the symbolizer is disabled and thus no symbol
5-
# names can be used.
6-
# See https://github.com/google/sanitizers/issues/1364
75

86
# -fsanitize=integer suppressions
97
# ===============================
108
# Dependencies
119
# ------------
1210
# Suppressions in dependencies that are developed outside this repository.
1311
unsigned-integer-overflow:*/include/c++/
14-
# unsigned-integer-overflow in FuzzedDataProvider's ConsumeIntegralInRange
15-
unsigned-integer-overflow:FuzzedDataProvider.h
12+
unsigned-integer-overflow:FuzzedDataProvider::ConsumeIntegralInRange
1613
unsigned-integer-overflow:leveldb/
1714
unsigned-integer-overflow:minisketch/
1815
unsigned-integer-overflow:test/fuzz/crypto_diff_fuzz_chacha20.cpp

0 commit comments

Comments
 (0)