aboutsummaryrefslogtreecommitdiff
path: root/src/test/fuzz/pow.cpp
diff options
context:
space:
mode:
authorAndrew Poelstra <apoelstra@wpsoftware.net>2021-10-25 19:48:22 +0000
committerAndrew Poelstra <apoelstra@wpsoftware.net>2021-11-12 19:51:55 +0000
commit214d9055acdd72189a2f415477ce472ca8db4191 (patch)
treeceb70a5558b22371e5c3c4eddb11e967c4249192 /src/test/fuzz/pow.cpp
parent22a90186496aea8025316bc5616905ffcf1aeb29 (diff)
downloadbitcoin-214d9055acdd72189a2f415477ce472ca8db4191.tar.xz
fuzz: replace every fuzzer-controlled loop with a LIMITED_WHILE loop
Blindly chose a cap of 10000 iterations for every loop, except for the two in script_ops.cpp and scriptnum_ops.cpp which appeared to (sometimes) be deserializing individual bytes; capped those to one million to ensure that sometimes we try working with massive scripts. There was also one fuzzer-controlled loop in timedata.cpp which was already capped, so I left that alone. git grep 'while (fuzz' should now run clean except for timedata.cpp
Diffstat (limited to 'src/test/fuzz/pow.cpp')
-rw-r--r--src/test/fuzz/pow.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/fuzz/pow.cpp b/src/test/fuzz/pow.cpp
index 47b4323e81..1123c8c170 100644
--- a/src/test/fuzz/pow.cpp
+++ b/src/test/fuzz/pow.cpp
@@ -27,7 +27,7 @@ FUZZ_TARGET_INIT(pow, initialize_pow)
std::vector<CBlockIndex> blocks;
const uint32_t fixed_time = fuzzed_data_provider.ConsumeIntegral<uint32_t>();
const uint32_t fixed_bits = fuzzed_data_provider.ConsumeIntegral<uint32_t>();
- while (fuzzed_data_provider.remaining_bytes() > 0) {
+ LIMITED_WHILE(fuzzed_data_provider.remaining_bytes() > 0, 10000) {
const std::optional<CBlockHeader> block_header = ConsumeDeserializable<CBlockHeader>(fuzzed_data_provider);
if (!block_header) {
continue;