diff options
| author | Wladimir J. van der Laan <[email protected]> | 2017-09-20 19:02:23 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2017-09-20 19:07:40 +0200 |
| commit | 551d7bf604fb4615d64b8b644aefa160dadc34c4 (patch) | |
| tree | 99579b021c56fd230fa2eed5880015e56714befd /src/wallet/wallet.cpp | |
| parent | Merge #11305: [doc] Update release notes and manpages for 0.16 (diff) | |
| parent | Document assumptions that are being made to avoid NULL pointer dereferences (diff) | |
| download | discoin-551d7bf604fb4615d64b8b644aefa160dadc34c4.tar.xz discoin-551d7bf604fb4615d64b8b644aefa160dadc34c4.zip | |
Merge #11132: Document assumptions that are being made to avoid NULL pointer dereferences
fdc3293 Document assumptions that are being made to avoid NULL pointer dereferences (practicalswift)
Pull request description:
Document assumptions (via `assert(…)`:s) that are being made avoid `NULL` pointer dereferences.
Rationale:
* Make it clear to human reviewers and non-human static analyzers that what might look like potential `NULL` pointer dereferences are written the way they are intentionally (these cases are currently flagged by various static analyzers).
Tree-SHA512: b424328195e2680e1e4ec546298f718c49e5ad182147dc004de580693db1b50eec4065e1c4f232bdb302baa12954265a50ba21cb5ba4ff30248535b2de778672
Diffstat (limited to 'src/wallet/wallet.cpp')
| -rw-r--r-- | src/wallet/wallet.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index cbc045fa1..fd2c1dfbe 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -519,6 +519,7 @@ void CWallet::SyncMetaData(std::pair<TxSpends::iterator, TxSpends::iterator> ran const uint256& hash = it->second; CWalletTx* copyTo = &mapWallet[hash]; if (copyFrom == copyTo) continue; + assert(copyFrom && "Oldest wallet transaction in range assumed to have been found."); if (!copyFrom->IsEquivalentTo(*copyTo)) continue; copyTo->mapValue = copyFrom->mapValue; copyTo->vOrderForm = copyFrom->vOrderForm; |