diff options
| author | Samuel Dobson <[email protected]> | 2020-01-15 22:10:39 +1300 |
|---|---|---|
| committer | Samuel Dobson <[email protected]> | 2020-01-15 22:11:33 +1300 |
| commit | ac61ec9da6793f00b29ba11f784b9b1c3ae662e9 (patch) | |
| tree | fb898950ce506e21fcd5d41484c2d4228e5a4c02 /src/httpserver.cpp | |
| parent | Merge #17923: refactor: Use PACKAGE_NAME in GUI modal overlay and bitcoin-wallet (diff) | |
| parent | wallet: Reset reused transactions cache (diff) | |
| download | discoin-ac61ec9da6793f00b29ba11f784b9b1c3ae662e9.tar.xz discoin-ac61ec9da6793f00b29ba11f784b9b1c3ae662e9.zip | |
Merge #17843: wallet: Reset reused transactions cache
6fc554f591d8ea1681b8bb25aa12da8d4f023f66 wallet: Reset reused transactions cache (Fabian Jahr)
Pull request description:
Fixes #17603 (together with #17824)
`getbalances` is using the cache within `GetAvailableCredit` under certain conditions [here](https://github.com/bitcoin/bitcoin/blob/35fff5be60e853455abc24713481544e91adfedb/src/wallet/wallet.cpp#L1826). For a wallet with `avoid_reuse` activated this can lead to inconsistent reporting of `used` transactions/balances between `getbalances` and `listunspent` as pointed out in #17603. When an address is reused before the first transaction is spending from this address, the cache is not updated even after the transaction is sent. This means the remaining outputs at the reused address are not showing up as `used` in `getbalances`.
With this change, any newly incoming transaction belonging to the wallet marks all the other outputs at the same address as dirty.
ACKs for top commit:
kallewoof:
Code review re-ACK 6fc554f591d8ea1681b8bb25aa12da8d4f023f66
promag:
ACK 6fc554f591d8ea1681b8bb25aa12da8d4f023f66.
achow101:
Re-ACK 6fc554f591d8ea1681b8bb25aa12da8d4f023f66
meshcollider:
Code review ACK 6fc554f591d8ea1681b8bb25aa12da8d4f023f66
Tree-SHA512: c4cad2c752176d16d77b4a4202291d20baddf9f27250896a40274d74a6945e0f6b34be04c2f9b1b2e756d3ac669b794969df8f82a98e0b16f10e92f276649ea2
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions