Merge pull request #7960

2e8936f rpc: Fix get_transactions failing when not found (Nathan Dorfman)
This commit is contained in:
luigi1111 2021-10-11 12:59:32 -05:00
commit a6367693c7
No known key found for this signature in database
GPG key ID: F4ACA0183641E010

View file

@ -972,14 +972,26 @@ namespace cryptonote
LOG_PRINT_L2("Found " << found_in_pool << "/" << vh.size() << " transactions in the pool"); LOG_PRINT_L2("Found " << found_in_pool << "/" << vh.size() << " transactions in the pool");
} }
std::vector<std::string>::const_iterator txhi = req.txs_hashes.begin(); CHECK_AND_ASSERT_MES(txs.size() + missed_txs.size() == vh.size(), false, "mismatched number of txs");
std::vector<crypto::hash>::const_iterator vhi = vh.begin();
auto txhi = req.txs_hashes.cbegin();
auto vhi = vh.cbegin();
auto missedi = missed_txs.cbegin();
for(auto& tx: txs) for(auto& tx: txs)
{ {
res.txs.push_back(COMMAND_RPC_GET_TRANSACTIONS::entry()); res.txs.push_back(COMMAND_RPC_GET_TRANSACTIONS::entry());
COMMAND_RPC_GET_TRANSACTIONS::entry &e = res.txs.back(); COMMAND_RPC_GET_TRANSACTIONS::entry &e = res.txs.back();
while (missedi != missed_txs.end() && *missedi == *vhi)
{
++vhi;
++txhi;
++missedi;
}
crypto::hash tx_hash = *vhi++; crypto::hash tx_hash = *vhi++;
CHECK_AND_ASSERT_MES(tx_hash == std::get<0>(tx), false, "mismatched tx hash");
e.tx_hash = *txhi++; e.tx_hash = *txhi++;
e.prunable_hash = epee::string_tools::pod_to_hex(std::get<2>(tx)); e.prunable_hash = epee::string_tools::pod_to_hex(std::get<2>(tx));
if (req.split || req.prune || std::get<3>(tx).empty()) if (req.split || req.prune || std::get<3>(tx).empty())