Skip to content

Commit

Permalink
Merge bitcoin#12860: Add testmempoolaccept to release-notes, Add miss…
Browse files Browse the repository at this point in the history
…ing const

fafcad3 doc: Add testmempoolaccept to release-notes (MarcoFalke)

Pull request description:

  Some fixups for bitcoin#11742:

  * Add release notes for the new rpc
  * Fix a typo in the original pull
  * Make the mempool reference passed to `CheckInputsFromMempoolAndCache` const, since that function is called before we return from ATMP and we must not modify the mempool.

Tree-SHA512: 72c459ba69f7698a69c91d2592f10f7fb1864846c7d8c525050d48286f92ba5ec5fe554c54235b52fbd9a8f00226c526ad84584641ec39084e1a1310a261510d
  • Loading branch information
laanwj authored and kwvg committed May 25, 2021
1 parent 64bbcbd commit 344b55c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/rpc/rawtransaction.cpp
Expand Up @@ -1222,7 +1222,7 @@ UniValue testmempoolaccept(const JSONRPCRequest& request)
{
LOCK(cs_main);
test_accept_res = AcceptToMemoryPool(mempool, state, std::move(tx), &missing_inputs,
false /* bypass_limits */, max_raw_tx_fee, /* test_accpet */ true);
false /* bypass_limits */, max_raw_tx_fee, /* test_accept */ true);
}
result_0.pushKV("allowed", test_accept_res);
if (!test_accept_res) {
Expand Down
2 changes: 1 addition & 1 deletion src/validation.cpp
Expand Up @@ -591,7 +591,7 @@ void UpdateMempoolForReorg(DisconnectedBlockTransactions &disconnectpool, bool f

// Used to avoid mempool polluting consensus critical paths if CCoinsViewMempool
// were somehow broken and returning the wrong scriptPubKeys
static bool CheckInputsFromMempoolAndCache(const CTransaction& tx, CValidationState &state, const CCoinsViewCache &view, CTxMemPool& pool,
static bool CheckInputsFromMempoolAndCache(const CTransaction& tx, CValidationState& state, const CCoinsViewCache& view, const CTxMemPool& pool,
unsigned int flags, bool cacheSigStore, PrecomputedTransactionData& txdata) {
AssertLockHeld(cs_main);

Expand Down

0 comments on commit 344b55c

Please sign in to comment.