Files
bitcoin/src/test/fuzz/private_broadcast.cpp
Gregory Sanders 5aea3d0373 private broadcast: limit outstanding txs to count of 10,000
Add a belt-and-suspenders feature, limit the amount of
memory and cpu possible when unlucky or simply misconfigured.
The worst case limit is roughly 400kB * 10,000 = 4GB, regardless
of usage pattern.

Before this change, sheer volume of broadcasts, mismatches in
standardness rules, or simply fee mismatches may result in unbounded
growth of memory usage. As the feature may be expanded in
the future, explicit bounds helps reasoning going forward.
2026-06-30 14:53:59 -04:00

209 lines
8.4 KiB
C++

// Copyright (c) 2025-present The Bitcoin Core developers
// Distributed under the MIT software license, see the accompanying
// file COPYING or http://www.opensource.org/licenses/mit-license.php.
#include <consensus/tx_check.h>
#include <consensus/validation.h>
#include <net.h>
#include <primitives/transaction.h>
#include <private_broadcast.h>
#include <test/fuzz/FuzzedDataProvider.h>
#include <test/fuzz/fuzz.h>
#include <test/fuzz/util.h>
#include <test/fuzz/util/net.h>
#include <test/util/setup_common.h>
#include <test/util/time.h>
#include <util/overflow.h>
#include <util/time.h>
#include <unordered_set>
struct CTransactionRefHash {
size_t operator()(const CTransactionRef& tx) const
{
return static_cast<size_t>(tx->GetWitnessHash().ToUint256().GetUint64(0));
}
};
struct CTransactionRefComp {
bool operator()(const CTransactionRef& a, const CTransactionRef& b) const
{
return a->GetWitnessHash() == b->GetWitnessHash();
}
};
FUZZ_TARGET(private_broadcast)
{
SeedRandomStateForTest(SeedRand::ZEROS);
FuzzedDataProvider fdp(buffer.data(), buffer.size());
FakeNodeClock clock_ctx{ConsumeTime(fdp)};
const size_t cap{fdp.ConsumeIntegralInRange<size_t>(1, 12)};
PrivateBroadcast pb{cap};
// Random transaction that the test generated and passed to Add(). Trimmed when Remove() is called.
// The values are the number of times a transaction was picked for sending.
std::unordered_map<CTransactionRef, size_t, CTransactionRefHash, CTransactionRefComp> transactions;
// Ids of nodes that were passed to PickTxForSend(). Trimmed when Remove() is called.
std::unordered_set<NodeId> nodes_sent_to;
// A subset of `nodes_sent_to`, node ids passed to NodeConfirmedReception(). Trimmed when Remove() is called.
std::unordered_set<NodeId> nodes_that_confirmed_reception;
NodeId next_nodeid{0}; // Generate unique node ids.
const auto ExistentOrNewNodeId = [&next_nodeid, &fdp](){
if (next_nodeid == 0 || fdp.ConsumeBool()) {
return next_nodeid++;
}
return fdp.ConsumeIntegralInRange<NodeId>(0, next_nodeid - 1);
};
LIMITED_WHILE(fdp.ConsumeBool(), 10000) {
CallOneOf(
fdp,
[&] { // Add()
CTransactionRef tx;
if (transactions.empty() || fdp.ConsumeBool()) {
tx = MakeTransactionRef(ConsumeTransaction(fdp, std::nullopt));
} else {
tx = PickIterator(fdp, transactions)->first;
}
const bool present_before{transactions.contains(tx)};
const auto res{pb.Add(tx)};
if (present_before) {
Assert(res == PrivateBroadcast::AddResult::AlreadyPresent);
} else if (transactions.size() >= cap) {
Assert(res == PrivateBroadcast::AddResult::QueueFull);
} else {
Assert(res == PrivateBroadcast::AddResult::Added);
transactions.emplace(tx, 0);
}
},
[&] { // Remove()
if (transactions.empty()) {
return;
}
const auto transactions_it{PickIterator(fdp, transactions)};
const CTransactionRef& tx{transactions_it->first};
size_t num_nodes_that_confirmed_tx{0};
// Remove relevant entries from nodes_sent_to[] and nodes_that_confirmed_reception[] if any.
for (auto it = nodes_sent_to.begin(); it != nodes_sent_to.end();) {
const NodeId nodeid{*it};
const auto opt_tx_for_node{pb.GetTxForNode(nodeid)};
if (opt_tx_for_node.has_value() && opt_tx_for_node.value() == tx) {
it = nodes_sent_to.erase(it);
if (nodes_that_confirmed_reception.erase(nodeid) > 0) {
++num_nodes_that_confirmed_tx;
}
} else {
++it;
}
}
const auto opt_num_confirmed{pb.Remove(tx)};
Assert(opt_num_confirmed.has_value());
Assert(opt_num_confirmed.value() == num_nodes_that_confirmed_tx);
Assert(!pb.Remove(tx).has_value());
transactions.erase(transactions_it);
},
[&] { // PickTxForSend()
// Only give pristine node ids to PickTxForSend() as required.
const NodeId will_send_to_nodeid{next_nodeid++};
const CService will_send_to_address{ConsumeService(fdp)};
const auto opt_tx{pb.PickTxForSend(will_send_to_nodeid, will_send_to_address)};
if (opt_tx.has_value()) {
Assert(transactions.contains(opt_tx.value()));
// "Number of times picked for sending" is the primary key in Priority's comparison
// (fewest sends = highest priority), so PickTxForSend() must return a transaction
// with the minimum send count of any in the queue. Ties are broken by state we
// don't model, so only check this key.
const size_t min_picked{std::ranges::min_element(
transactions, {}, [](const auto& el) { return el.second; })->second};
const auto picked_it{transactions.find(opt_tx.value())};
Assert(picked_it != transactions.end());
Assert(picked_it->second == min_picked); // picked the least-sent transaction
++picked_it->second; // PickTxForSend() recorded exactly one send
const auto& [_, inserted]{nodes_sent_to.emplace(will_send_to_nodeid)};
Assert(inserted);
} else {
Assert(transactions.empty());
}
},
[&] { // GetTxForNode()
const NodeId nodeid{ExistentOrNewNodeId()};
const auto opt_tx{pb.GetTxForNode(nodeid)};
if (nodes_sent_to.contains(nodeid)) {
Assert(opt_tx.has_value());
Assert(transactions.contains(opt_tx.value()));
} else {
Assert(!opt_tx.has_value());
}
},
[&] { // NodeConfirmedReception()
const NodeId nodeid{ExistentOrNewNodeId()};
pb.NodeConfirmedReception(nodeid);
if (nodes_sent_to.contains(nodeid)) {
// nodeid was previously passed to PickTxForSend(), so NodeConfirmedReception()
// must have changed the internal state. Remember this to later check that
// DidNodeConfirmReception() works correctly.
nodes_that_confirmed_reception.emplace(nodeid);
}
},
[&] { // DidNodeConfirmReception()
const NodeId nodeid{ExistentOrNewNodeId()};
const bool confirmed{pb.DidNodeConfirmReception(nodeid)};
if (nodes_that_confirmed_reception.contains(nodeid)) {
Assert(confirmed);
} else {
Assert(!confirmed);
}
},
[&] { // HavePendingTransactions()
if (pb.HavePendingTransactions()) {
Assert(!transactions.empty());
} else {
Assert(transactions.empty());
}
},
[&] { // GetStale()
const auto stale{pb.GetStale()};
Assert(stale.size() <= transactions.size());
for (const auto& stale_tx : stale) {
Assert(transactions.contains(stale_tx));
}
},
[&] { // GetBroadcastInfo()
const auto all_broadcast_info{pb.GetBroadcastInfo()};
Assert(all_broadcast_info.size() == transactions.size());
for (const auto& info : all_broadcast_info) {
const auto it{transactions.find(info.tx)};
Assert(it != transactions.end());
Assert(info.peers.size() == it->second); // exactly the sends we recorded
}
},
[&] {
clock_ctx.set(ConsumeTime(fdp));
});
}
}