mirror of
https://git.wownero.com/wownero/wownero.git
synced 2024-08-15 01:03:23 +00:00
Merge pull request #5273
bc61b5ca
wallet2: don't store 0 amount outputs, they'll fail to be spent (moneromooo-monero)
This commit is contained in:
commit
503b2fd964
1 changed files with 7 additions and 1 deletions
|
@ -1495,11 +1495,17 @@ void wallet2::scan_output(const cryptonote::transaction &tx, bool miner_tx, cons
|
||||||
}
|
}
|
||||||
|
|
||||||
THROW_WALLET_EXCEPTION_IF(std::find(outs.begin(), outs.end(), i) != outs.end(), error::wallet_internal_error, "Same output cannot be added twice");
|
THROW_WALLET_EXCEPTION_IF(std::find(outs.begin(), outs.end(), i) != outs.end(), error::wallet_internal_error, "Same output cannot be added twice");
|
||||||
outs.push_back(i);
|
|
||||||
if (tx_scan_info.money_transfered == 0 && !miner_tx)
|
if (tx_scan_info.money_transfered == 0 && !miner_tx)
|
||||||
{
|
{
|
||||||
tx_scan_info.money_transfered = tools::decodeRct(tx.rct_signatures, tx_scan_info.received->derivation, i, tx_scan_info.mask, m_account.get_device());
|
tx_scan_info.money_transfered = tools::decodeRct(tx.rct_signatures, tx_scan_info.received->derivation, i, tx_scan_info.mask, m_account.get_device());
|
||||||
}
|
}
|
||||||
|
if (tx_scan_info.money_transfered == 0)
|
||||||
|
{
|
||||||
|
MERROR("Invalid output amount, skipping");
|
||||||
|
tx_scan_info.error = true;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
outs.push_back(i);
|
||||||
THROW_WALLET_EXCEPTION_IF(tx_money_got_in_outs[tx_scan_info.received->index] >= std::numeric_limits<uint64_t>::max() - tx_scan_info.money_transfered,
|
THROW_WALLET_EXCEPTION_IF(tx_money_got_in_outs[tx_scan_info.received->index] >= std::numeric_limits<uint64_t>::max() - tx_scan_info.money_transfered,
|
||||||
error::wallet_internal_error, "Overflow in received amounts");
|
error::wallet_internal_error, "Overflow in received amounts");
|
||||||
tx_money_got_in_outs[tx_scan_info.received->index] += tx_scan_info.money_transfered;
|
tx_money_got_in_outs[tx_scan_info.received->index] += tx_scan_info.money_transfered;
|
||||||
|
|
Loading…
Reference in a new issue