diff options
author | MarcoFalke <falke.marco@gmail.com> | 2022-02-02 15:00:19 +0100 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2022-02-02 15:00:22 +0100 |
commit | 219d728fcbde8c313940788838afa46c2fb88762 (patch) | |
tree | e27c8bbf1d7545047675e097bf38d0548430861a /test/sanitizer_suppressions | |
parent | a41976ab770d4453ccb043f72e48a6eb15a7106a (diff) | |
parent | fad84a25956ec081f22aebbda309d168a3dc0004 (diff) | |
download | bitcoin-219d728fcbde8c313940788838afa46c2fb88762.tar.xz |
Merge bitcoin/bitcoin#24219: Fix implicit-integer-sign-change in bloom
fad84a25956ec081f22aebbda309d168a3dc0004 refactor: Fixup uint64_t-cast style in touched line (MarcoFalke)
fa041878de786f5be74ec74a06ec407c99ca8656 Fix implicit-integer-sign-change in bloom (MarcoFalke)
Pull request description:
Signed values don't really make sense when using `std::vector::operator[]`.
Fix that and remove the suppression.
ACKs for top commit:
PastaPastaPasta:
utACK fad84a25956ec081f22aebbda309d168a3dc0004
theStack:
Code-review ACK fad84a25956ec081f22aebbda309d168a3dc0004
Tree-SHA512: 7139dd9aa098c41e4af1b6e63dd80e71a92b0a98062d1676b01fe550ffa8e21a5f84a578afa7a536d70dad1b8a5017625e3a9e2dda6f864b452ec77b130ddf2a
Diffstat (limited to 'test/sanitizer_suppressions')
-rw-r--r-- | test/sanitizer_suppressions/ubsan | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test/sanitizer_suppressions/ubsan b/test/sanitizer_suppressions/ubsan index a049693ee1..ec13acb689 100644 --- a/test/sanitizer_suppressions/ubsan +++ b/test/sanitizer_suppressions/ubsan @@ -60,7 +60,6 @@ unsigned-integer-overflow:util/strencodings.cpp unsigned-integer-overflow:validation.cpp implicit-integer-sign-change:addrman.h implicit-integer-sign-change:bech32.cpp -implicit-integer-sign-change:common/bloom.cpp implicit-integer-sign-change:coins.h implicit-integer-sign-change:compat/stdin.cpp implicit-integer-sign-change:compressor.h |