Skip to content

Commit 31e5116

Browse files
committed
Merge #20674: fuzz: Call SendMessages after ProcessMessage to increase coverage
fa09f97 fuzz: Call SendMessages after ProcessMessage to increase coverage (MarcoFalke) Pull request description: ACKs for top commit: practicalswift: Tested ACK fa09f97 dhruv: tACK fa09f97 Crypt-iQ: cr ACK fa09f97 sipa: utACK fa09f97 Tree-SHA512: 87c52aa38f902c4f6c9c2380f486a3ab21edc0e21e48bb619cdb67cfd698154cc57b170eef31fc940c0bb2c878e155847de03fc6e4cd85bed25f10c4f80c747b
2 parents 02cf20b + fa09f97 commit 31e5116

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

src/test/fuzz/process_message.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ void fuzz_target(const std::vector<uint8_t>& buffer, const std::string& LIMIT_TO
7575
GetTime<std::chrono::microseconds>(), std::atomic<bool>{false});
7676
} catch (const std::ios_base::failure&) {
7777
}
78+
{
79+
LOCK(p2p_node.cs_sendProcessing);
80+
g_setup->m_node.peerman->SendMessages(&p2p_node);
81+
}
7882
SyncWithValidationInterfaceQueue();
7983
LOCK2(::cs_main, g_cs_orphans); // See init.cpp for rationale for implicit locking order requirement
8084
g_setup->m_node.connman->StopNodes();

src/test/fuzz/process_messages.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,10 @@ FUZZ_TARGET_INIT(process_messages, initialize_process_messages)
7979
connman.ProcessMessagesOnce(random_node);
8080
} catch (const std::ios_base::failure&) {
8181
}
82+
{
83+
LOCK(random_node.cs_sendProcessing);
84+
g_setup->m_node.peerman->SendMessages(&random_node);
85+
}
8286
}
8387
SyncWithValidationInterfaceQueue();
8488
LOCK2(::cs_main, g_cs_orphans); // See init.cpp for rationale for implicit locking order requirement

0 commit comments

Comments
 (0)