diff options
| author | Carl Dong <contact@carldong.me> | 2021-03-17 16:58:22 -0400 |
|---|---|---|
| committer | Carl Dong <contact@carldong.me> | 2021-03-30 13:52:22 -0400 |
| commit | 88aead263c61d86e5f836028f517cfbf2a575498 (patch) | |
| tree | be96a4eea8c07f4bcafd709dd791df85fec39045 | |
| parent | 1dd8ed7a8491e51b76eeb236b15b794d9254f674 (diff) | |
node: Avoid potential UB by asserting assumptions
| -rw-r--r-- | src/node/coin.cpp | 1 | ||||
| -rw-r--r-- | src/node/transaction.cpp | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/src/node/coin.cpp b/src/node/coin.cpp index 263dcff657..23d4fa2aae 100644 --- a/src/node/coin.cpp +++ b/src/node/coin.cpp @@ -11,6 +11,7 @@ void FindCoins(const NodeContext& node, std::map<COutPoint, Coin>& coins) { assert(node.mempool); + assert(node.chainman); LOCK2(cs_main, node.mempool->cs); assert(std::addressof(::ChainstateActive()) == std::addressof(node.chainman->ActiveChainstate())); CCoinsViewCache& chain_view = node.chainman->ActiveChainstate().CoinsTip(); diff --git a/src/node/transaction.cpp b/src/node/transaction.cpp index f47e85aceb..691b2791d7 100644 --- a/src/node/transaction.cpp +++ b/src/node/transaction.cpp @@ -38,6 +38,7 @@ TransactionError BroadcastTransaction(NodeContext& node, const CTransactionRef t bool callback_set = false; { // cs_main scope + assert(node.chainman); LOCK(cs_main); assert(std::addressof(::ChainstateActive()) == std::addressof(node.chainman->ActiveChainstate())); // If the transaction is already confirmed in the chain, don't do anything |