Skip to content

Commit 6eafa81

Browse files
author
MarcoFalke
committed
Merge bitcoin#22267: fuzz: Speed up crypto fuzz target
fa483e9 fuzz: Speed up crypto fuzz target (MarcoFalke) Pull request description: May fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34962 Similar solution to bitcoin#22005 ACKs for top commit: practicalswift: cr ACK fa483e9: patch looks correct and rationale makes sense Tree-SHA512: 3788cf9f6ba0f7a0a217cd3a6a825839689425e99e4d6d657981d291a001b0da7c5abb50a68b4ee1c2a8300b87fb92e4e3ccc1171907792b40251e467c33bd53
2 parents dd24567 + fa483e9 commit 6eafa81

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/test/fuzz/crypto.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@
1919

2020
FUZZ_TARGET(crypto)
2121
{
22+
// Hashing is expensive with sanitizers enabled, so limit the number of
23+
// calls
24+
int limit_max_ops{30};
25+
2226
FuzzedDataProvider fuzzed_data_provider{buffer.data(), buffer.size()};
2327
std::vector<uint8_t> data = ConsumeRandomLengthByteVector(fuzzed_data_provider);
2428
if (data.empty()) {
@@ -36,7 +40,7 @@ FUZZ_TARGET(crypto)
3640
SHA3_256 sha3;
3741
CSipHasher sip_hasher{fuzzed_data_provider.ConsumeIntegral<uint64_t>(), fuzzed_data_provider.ConsumeIntegral<uint64_t>()};
3842

39-
while (fuzzed_data_provider.ConsumeBool()) {
43+
while (--limit_max_ops >= 0 && fuzzed_data_provider.ConsumeBool()) {
4044
CallOneOf(
4145
fuzzed_data_provider,
4246
[&] {

0 commit comments

Comments
 (0)