diff options
| author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-10-08 13:09:18 +0000 |
|---|---|---|
| committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-10-29 16:22:55 +0000 |
| commit | cbe0122bf18d702e1fb7c383ff3681544bb74005 (patch) | |
| tree | 5825899f16dc9a3bb6179904520a70acb82e831e /src/wallet | |
| parent | Merge pull request #4721 (diff) | |
| download | monero-cbe0122bf18d702e1fb7c383ff3681544bb74005.tar.xz | |
wallet2: initialize amount to 0 in tx_scan_info_t ctor
It seems the more prudent thing to do here. It will not catch
attempts to use that value before it is initialized when using
ASAN or valgrind, but in a case where it does, it will have
smaller repercussions.
So it seems appropriate in this particular case.
Coverity 182498
Diffstat (limited to 'src/wallet')
| -rw-r--r-- | src/wallet/wallet2.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wallet/wallet2.h b/src/wallet/wallet2.h index 680196f01..caf77c60f 100644 --- a/src/wallet/wallet2.h +++ b/src/wallet/wallet2.h @@ -229,7 +229,7 @@ namespace tools bool error; boost::optional<cryptonote::subaddress_receive_info> received; - tx_scan_info_t(): money_transfered(0), error(true) {} + tx_scan_info_t(): amount(0), money_transfered(0), error(true) {} }; struct transfer_details |