Skip to content

Commit e8c7434

Browse files
author
MarcoFalke
committed
Merge #13683: wallet: Introduce assertion to document the assumption that cache and cache_used are always set in tandem
d063303 wallet: Avoid potential null pointer dereference in CWalletTx::GetAvailableCredit(...) (practicalswift) Pull request description: Avoid potential null pointer dereference in `CWalletTx::GetAvailableCredit(...)`. Introduced in 4279da4. Tree-SHA512: 92d1da9682f0bab11f6f96395ca30549331b0a056cbceb7e1a7f98b3d17d10082aaeed0907cafd46f4164b0e0f4b77e01f78bfd7d24e0503a66c6942ae842aa5
2 parents eeae471 + d063303 commit e8c7434

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/wallet/wallet.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1977,6 +1977,7 @@ CAmount CWalletTx::GetAvailableCredit(bool fUseCache, const isminefilter& filter
19771977

19781978
if (cache) {
19791979
*cache = nCredit;
1980+
assert(cache_used);
19801981
*cache_used = true;
19811982
}
19821983
return nCredit;

0 commit comments

Comments
 (0)