Skip to content

Commit fa98455

Browse files
author
MarcoFalke
committed
fuzz: Set ignore_incoming_txs in p2p_headers_presync
This avoids non-determistic code paths. Without this patch, the tool would report a diff: cargo run --manifest-path ./contrib/devtools/deterministic-fuzz-coverage/Cargo.toml -- $PWD/bld-cmake/ $PWD/../qa-assets/fuzz_corpora/ p2p_headers_presync 32 ... - 5371| 393| peer.m_next_send_feefilter = current_time + m_rng.randrange<std::chrono::microseconds>(MAX_FEEFILTER_CHANGE_DELAY); - 5372| 393| } + 5371| 396| peer.m_next_send_feefilter = current_time + m_rng.randrange<std::chrono::microseconds>(MAX_FEEFILTER_CHANGE_DELAY); + 5372| 396| } 5373| 16.2k|} ...
1 parent faf2e23 commit fa98455

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/test/fuzz/p2p_headers_presync.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ class HeadersSyncSetup : public TestingSetup
3131
PeerManager::Options peerman_opts;
3232
node::ApplyArgsManOptions(*m_node.args, peerman_opts);
3333
peerman_opts.max_headers_result = FUZZ_MAX_HEADERS_RESULTS;
34+
// No txs are relayed. Disable irrelevant and possibly
35+
// non-deterministic code paths.
36+
peerman_opts.ignore_incoming_txs = true;
3437
m_node.peerman = PeerManager::make(*m_node.connman, *m_node.addrman,
3538
m_node.banman.get(), *m_node.chainman,
3639
*m_node.mempool, *m_node.warnings, peerman_opts);

0 commit comments

Comments
 (0)