fuzz: re-enable prioritisetransaction & analyzepsbt RPC

This commit is contained in:
MarcoFalke 2023-04-13 20:36:25 +02:00
parent 19764dc143
commit faa7144d3c
No known key found for this signature in database

View File

@ -70,7 +70,6 @@ const std::vector<std::string> RPC_COMMANDS_NOT_SAFE_FOR_FUZZING{
"addconnection", // avoid DNS lookups "addconnection", // avoid DNS lookups
"addnode", // avoid DNS lookups "addnode", // avoid DNS lookups
"addpeeraddress", // avoid DNS lookups "addpeeraddress", // avoid DNS lookups
"analyzepsbt", // avoid signed integer overflow in CFeeRate::GetFee(unsigned long) (https://github.com/bitcoin/bitcoin/issues/20607)
"dumptxoutset", // avoid writing to disk "dumptxoutset", // avoid writing to disk
"dumpwallet", // avoid writing to disk "dumpwallet", // avoid writing to disk
"echoipc", // avoid assertion failure (Assertion `"EnsureAnyNodeContext(request.context).init" && check' failed.) "echoipc", // avoid assertion failure (Assertion `"EnsureAnyNodeContext(request.context).init" && check' failed.)
@ -79,7 +78,6 @@ const std::vector<std::string> RPC_COMMANDS_NOT_SAFE_FOR_FUZZING{
"gettxoutproof", // avoid prohibitively slow execution "gettxoutproof", // avoid prohibitively slow execution
"importwallet", // avoid reading from disk "importwallet", // avoid reading from disk
"loadwallet", // avoid reading from disk "loadwallet", // avoid reading from disk
"prioritisetransaction", // avoid signed integer overflow in CTxMemPool::PrioritiseTransaction(uint256 const&, long const&) (https://github.com/bitcoin/bitcoin/issues/20626)
"savemempool", // disabled as a precautionary measure: may take a file path argument in the future "savemempool", // disabled as a precautionary measure: may take a file path argument in the future
"setban", // avoid DNS lookups "setban", // avoid DNS lookups
"stop", // avoid shutdown state "stop", // avoid shutdown state
@ -87,6 +85,7 @@ const std::vector<std::string> RPC_COMMANDS_NOT_SAFE_FOR_FUZZING{
// RPC commands which are safe for fuzzing. // RPC commands which are safe for fuzzing.
const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{ const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{
"analyzepsbt",
"clearbanned", "clearbanned",
"combinepsbt", "combinepsbt",
"combinerawtransaction", "combinerawtransaction",
@ -112,9 +111,9 @@ const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{
"getblockchaininfo", "getblockchaininfo",
"getblockcount", "getblockcount",
"getblockfilter", "getblockfilter",
"getblockfrompeer", // when no peers are connected, no p2p message is sent
"getblockhash", "getblockhash",
"getblockheader", "getblockheader",
"getblockfrompeer", // when no peers are connected, no p2p message is sent
"getblockstats", "getblockstats",
"getblocktemplate", "getblocktemplate",
"getchaintips", "getchaintips",
@ -128,7 +127,6 @@ const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{
"getmempoolancestors", "getmempoolancestors",
"getmempooldescendants", "getmempooldescendants",
"getmempoolentry", "getmempoolentry",
"gettxspendingprevout",
"getmempoolinfo", "getmempoolinfo",
"getmininginfo", "getmininginfo",
"getnettotals", "getnettotals",
@ -141,6 +139,7 @@ const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{
"getrpcinfo", "getrpcinfo",
"gettxout", "gettxout",
"gettxoutsetinfo", "gettxoutsetinfo",
"gettxspendingprevout",
"help", "help",
"invalidateblock", "invalidateblock",
"joinpsbts", "joinpsbts",
@ -149,6 +148,7 @@ const std::vector<std::string> RPC_COMMANDS_SAFE_FOR_FUZZING{
"mockscheduler", "mockscheduler",
"ping", "ping",
"preciousblock", "preciousblock",
"prioritisetransaction",
"pruneblockchain", "pruneblockchain",
"reconsiderblock", "reconsiderblock",
"scanblocks", "scanblocks",