Skip to content

Commit a40978d

Browse files
committed
[fuzz] Assert that Peer.m_tx_relay.m_relay_txs has been set correctly
1 parent 0bca5f2 commit a40978d

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/test/fuzz/util.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ void FillNode(FuzzedDataProvider& fuzzed_data_provider, ConnmanTestMsg& connman,
225225
const ServiceFlags remote_services = ConsumeWeakEnum(fuzzed_data_provider, ALL_SERVICE_FLAGS);
226226
const NetPermissionFlags permission_flags = ConsumeWeakEnum(fuzzed_data_provider, ALL_NET_PERMISSION_FLAGS);
227227
const int32_t version = fuzzed_data_provider.ConsumeIntegralInRange<int32_t>(MIN_PEER_PROTO_VERSION, std::numeric_limits<int32_t>::max());
228-
const bool filter_txs = fuzzed_data_provider.ConsumeBool();
228+
const bool relay_txs{fuzzed_data_provider.ConsumeBool()};
229229

230230
const CNetMsgMaker mm{0};
231231

@@ -241,7 +241,7 @@ void FillNode(FuzzedDataProvider& fuzzed_data_provider, ConnmanTestMsg& connman,
241241
uint64_t{1}, // dummy nonce
242242
std::string{}, // dummy subver
243243
int32_t{}, // dummy starting_height
244-
filter_txs),
244+
relay_txs),
245245
};
246246

247247
(void)connman.ReceiveMsgFrom(node, msg_version);
@@ -255,6 +255,9 @@ void FillNode(FuzzedDataProvider& fuzzed_data_provider, ConnmanTestMsg& connman,
255255
assert(node.nVersion == version);
256256
assert(node.GetCommonVersion() == std::min(version, PROTOCOL_VERSION));
257257
assert(node.nServices == remote_services);
258+
CNodeStateStats statestats;
259+
assert(peerman.GetNodeStateStats(node.GetId(), statestats));
260+
assert(statestats.m_relay_txs == (relay_txs && !node.IsBlockOnlyConn()));
258261
node.m_permissionFlags = permission_flags;
259262
if (successfully_connected) {
260263
CSerializedNetMsg msg_verack{mm.Make(NetMsgType::VERACK)};

0 commit comments

Comments
 (0)