Skip to content

Commit 3ef756a

Browse files
committed
Remove txmempool implicit-integer-sign-change sanitizer suppressions
1 parent d2f6d2a commit 3ef756a

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/txmempool.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -370,7 +370,7 @@ void CTxMemPoolEntry::UpdateDescendantState(int32_t modifySize, CAmount modifyFe
370370
nSizeWithDescendants += modifySize;
371371
assert(nSizeWithDescendants > 0);
372372
nModFeesWithDescendants = SaturatingAdd(nModFeesWithDescendants, modifyFee);
373-
nCountWithDescendants += modifyCount;
373+
nCountWithDescendants += uint64_t(modifyCount);
374374
assert(int64_t(nCountWithDescendants) > 0);
375375
}
376376

@@ -379,7 +379,7 @@ void CTxMemPoolEntry::UpdateAncestorState(int32_t modifySize, CAmount modifyFee,
379379
nSizeWithAncestors += modifySize;
380380
assert(nSizeWithAncestors > 0);
381381
nModFeesWithAncestors = SaturatingAdd(nModFeesWithAncestors, modifyFee);
382-
nCountWithAncestors += modifyCount;
382+
nCountWithAncestors += uint64_t(modifyCount);
383383
assert(int64_t(nCountWithAncestors) > 0);
384384
nSigOpCostWithAncestors += modifySigOps;
385385
assert(int(nSigOpCostWithAncestors) >= 0);

test/sanitizer_suppressions/ubsan

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ implicit-integer-sign-change:prevector.h
5656
implicit-integer-sign-change:script/bitcoinconsensus.cpp
5757
implicit-integer-sign-change:script/interpreter.cpp
5858
implicit-integer-sign-change:serialize.h
59-
implicit-integer-sign-change:txmempool.cpp
6059
implicit-signed-integer-truncation:crypto/
6160
implicit-unsigned-integer-truncation:crypto/
6261
shift-base:arith_uint256.cpp

0 commit comments

Comments
 (0)