HomePhabricator

Add AssertLockHeld assertions in CWallet::ListCoins

Description

Add AssertLockHeld assertions in CWallet::ListCoins

Summary:

  • Add AssertLockHeld assertions in CWallet::ListCoins
  • Add EXCLUSIVE_LOCKS_REQUIRED to CWallet::ListCoins

Suggested by MarcoFalke <falke.marco@gmail.com> in
https://github.com/bitcoin/bitcoin/pull/10605#issuecomment-417643535

This is a backport of Core PR10605

Test Plan:

  • Build with clang and make sure there are no warning related to locks.
  • Build in debug mode and run the extended test suite.

Reviewers: #bitcoin_abc, jasonbcox

Reviewed By: #bitcoin_abc, jasonbcox

Differential Revision: https://reviews.bitcoinabc.org/D4170

Details

Provenance
Russell Yanofsky <russ@yanofsky.org>Authored on Jun 15 2017, 18:56
deadalnixCommitted on Sep 28 2019, 12:41
deadalnixPushed on Sep 28 2019, 13:46
Reviewer
Restricted Project
Differential Revision
D4170: Add AssertLockHeld assertions in CWallet::ListCoins
Parents
rSTAGINGaf3079ad32ef: Merge #13304: qa: Fix wallet_listreceivedby race
Branches
Unknown
Tags
Unknown