diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-02-07 14:03:49 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-02-07 14:03:56 +0100 |
| commit | fa65f26f4dd6135abb88410023f5ada753d5b257 (patch) | |
| tree | c8ea74b5485da862727094c08d8613b9e757823e /src/test/fuzz | |
| parent | 5034b7fa3b97c1cbaf93b337d462777f3e25d256 (diff) | |
| parent | fad81548fa03861c244397201d6b6e6cbf883c38 (diff) | |
| download | bitcoin-fa65f26f4dd6135abb88410023f5ada753d5b257.tar.xz | |
Merge bitcoin/bitcoin#24237: test: Avoid testing negative block heights
fad81548fa03861c244397201d6b6e6cbf883c38 test: Avoid testing negative block heights (MarcoFalke)
Pull request description:
A negative chain height is only used to denote an empty chain, not the height of any block.
So stop testing that and remove a suppression.
ACKs for top commit:
brunoerg:
crACK fad81548fa03861c244397201d6b6e6cbf883c38
Tree-SHA512: 0f9e91617dfb6ceda99831e6cf4b4bf0d951054957c159b1a05a178ab6090798fae7368edefe12800da24585bcdf7299ec3534f4d3bbf5ce6a6eca74dd3bb766
Diffstat (limited to 'src/test/fuzz')
| -rw-r--r-- | src/test/fuzz/coins_view.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/fuzz/coins_view.cpp b/src/test/fuzz/coins_view.cpp index 994b4b9e49..360dc00307 100644 --- a/src/test/fuzz/coins_view.cpp +++ b/src/test/fuzz/coins_view.cpp @@ -211,7 +211,7 @@ FUZZ_TARGET_INIT(coins_view, initialize_coins_view) return; } bool expected_code_path = false; - const int height = fuzzed_data_provider.ConsumeIntegral<int>(); + const int height{int(fuzzed_data_provider.ConsumeIntegral<uint32_t>() >> 1)}; const bool possible_overwrite = fuzzed_data_provider.ConsumeBool(); try { AddCoins(coins_view_cache, transaction, height, possible_overwrite); |