test: Suppress implicit-unsigned-integer-truncation:SaltedCoinsCacheHasher::operator()
What the developer wrote
Authored by MarcoFalke
test: Suppress implicit-unsigned-integer-truncation:SaltedCoinsCacheHasher::operator()
What changed, and why it matters
This commit adds a single line to a test-only sanitizer suppression file. It tells the UndefinedBehaviorSanitizer (UBSan) to ignore a specific unsigned integer truncation warning inside a hash function used by Bitcoin Core's coin cache. This is not a code fix and does not change runtime behavior in production builds. It merely reduces noise during automated testing with sanitizers.
No immediate action required. Treat as a test-hygiene change. If reviewing, confirm that the truncation in `SaltedCoinsCacheHasher::operator()` is intentional and harmless, and consider whether a code-level cast or comment would be preferable to a permanent suppression.
Security signals we found
Suppression of an undefined-behavior sanitizer warning
Warning concerns implicit unsigned integer truncation in a hash function
Change is confined to a test/CI sanitizer suppression file
Evidence from the diff
The diff adds implicit-unsigned-integer-truncation:SaltedCoinsCacheHasher::operator() to test/sanitizer_suppressions/ubsan. This suppresses a UBSan diagnostic for an intentional or benign truncation in SaltedCoinsCacheHasher::operator(). The file is only consumed by CI/test configurations that enable UBSan; production binaries are unaffected. No functional code is modified.
Changed components
test/sanitizer_suppressions/ubsanInspect captured patch +1 / −0
diff --git a/test/sanitizer_suppressions/ubsan b/test/sanitizer_suppressions/ubsan
index 063a8e31..d2102f0d 100644
--- a/test/sanitizer_suppressions/ubsan
+++ b/test/sanitizer_suppressions/ubsan
@@ -61,6 +61,7 @@ implicit-integer-sign-change:crypto/
implicit-integer-sign-change:TxConfirmStats::removeTx
implicit-integer-sign-change:EvalScript
implicit-signed-integer-truncation:crypto/
+implicit-unsigned-integer-truncation:SaltedCoinsCacheHasher::operator()
implicit-unsigned-integer-truncation:crypto/
shift-base:arith_uint256.cpp
shift-base:crypto/
Why this scored 17/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.