wallet2: do not remove known rings when a tx fails

Even if it fails, the ring composition is known to a potential
adversary, and so we should reuse the same ring next time
This commit is contained in:
moneromooo-monero 2020-01-09 12:48:57 +00:00
parent 907292dfb4
commit f507a43a23
No known key found for this signature in database
GPG key ID: 686F07454D6CEFC3

View file

@ -2907,7 +2907,6 @@ void wallet2::update_pool_state(std::vector<std::pair<cryptonote::transaction, b
pit->second.m_state = wallet2::unconfirmed_transfer_details::failed; pit->second.m_state = wallet2::unconfirmed_transfer_details::failed;
// the inputs aren't spent anymore, since the tx failed // the inputs aren't spent anymore, since the tx failed
remove_rings(pit->second.m_tx);
for (size_t vini = 0; vini < pit->second.m_tx.vin.size(); ++vini) for (size_t vini = 0; vini < pit->second.m_tx.vin.size(); ++vini)
{ {
if (pit->second.m_tx.vin[vini].type() == typeid(txin_to_key)) if (pit->second.m_tx.vin[vini].type() == typeid(txin_to_key))