diff options
| author | Ben Woosley <ben.woosley@gmail.com> | 2018-07-12 17:19:00 -0400 |
|---|---|---|
| committer | Ben Woosley <ben.woosley@gmail.com> | 2018-07-13 11:16:08 -0400 |
| commit | 89e70f9d7fe384ef9de4fa3828d4c80523290186 (patch) | |
| tree | 6b414840dd03987cddd9237864a98ecdf76ddfa1 /src | |
| parent | dcb154e5aad80e49ff41a7851604ac46f38cb167 (diff) | |
| download | bitcoin-89e70f9d7fe384ef9de4fa3828d4c80523290186.tar.xz | |
Fix that CWallet::AbandonTransaction would only traverse one level
Prior to this change, it would mark only the first layer of
child transactions abandoned, due to always following the input hashTx
rather than the current now tx.
Diffstat (limited to 'src')
| -rw-r--r-- | src/wallet/wallet.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index cdba6c6441..b8cbaeeec7 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -1146,7 +1146,7 @@ bool CWallet::AbandonTransaction(const uint256& hashTx) batch.WriteTx(wtx); NotifyTransactionChanged(this, wtx.GetHash(), CT_UPDATED); // Iterate over all its outputs, and mark transactions in the wallet that spend them abandoned too - TxSpends::const_iterator iter = mapTxSpends.lower_bound(COutPoint(hashTx, 0)); + TxSpends::const_iterator iter = mapTxSpends.lower_bound(COutPoint(now, 0)); while (iter != mapTxSpends.end() && iter->first.hash == now) { if (!done.count(iter->second)) { todo.insert(iter->second); |