rpc: reject wrong sized txid

Reporter requested credit to be given to Decred
This commit is contained in:
moneromooo-monero 2020-07-29 17:40:46 +00:00
parent 5b761d0186
commit 870a7b5201
No known key found for this signature in database
GPG Key ID: 686F07454D6CEFC3
1 changed files with 4 additions and 6 deletions

View File

@ -2453,14 +2453,13 @@ namespace cryptonote
{ {
for (const auto &str: req.txids) for (const auto &str: req.txids)
{ {
cryptonote::blobdata txid_data; crypto::hash txid;
if(!epee::string_tools::parse_hexstr_to_binbuff(str, txid_data)) if(!epee::string_tools::hex_to_pod(str, txid))
{ {
failed = true; failed = true;
} }
else else
{ {
crypto::hash txid = *reinterpret_cast<const crypto::hash*>(txid_data.data());
txids.push_back(txid); txids.push_back(txid);
} }
} }
@ -2805,15 +2804,14 @@ namespace cryptonote
res.status = ""; res.status = "";
for (const auto &str: req.txids) for (const auto &str: req.txids)
{ {
cryptonote::blobdata txid_data; crypto::hash txid;
if(!epee::string_tools::parse_hexstr_to_binbuff(str, txid_data)) if(!epee::string_tools::hex_to_pod(str, txid))
{ {
if (!res.status.empty()) res.status += ", "; if (!res.status.empty()) res.status += ", ";
res.status += std::string("invalid transaction id: ") + str; res.status += std::string("invalid transaction id: ") + str;
failed = true; failed = true;
continue; continue;
} }
crypto::hash txid = *reinterpret_cast<const crypto::hash*>(txid_data.data());
cryptonote::blobdata txblob; cryptonote::blobdata txblob;
if (m_core.get_pool_transaction(txid, txblob, relay_category::legacy)) if (m_core.get_pool_transaction(txid, txblob, relay_category::legacy))