Page MenuHomePhabricator

Add AssertLockHeld assertions in CWallet::ListCoins

Authored by deadalnix on Sep 27 2019, 16:14.


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

Suggested by MarcoFalke <> in

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.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Sep 27 2019, 16:14
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 27 2019, 16:14
deadalnix abandoned this revision.Sep 27 2019, 16:55
deadalnix reclaimed this revision.
deadalnix retitled this revision from Add EXCLUSIVE_LOCKS_REQUIRED to CWallet::ListCoins to Add AssertLockHeld assertions in CWallet::ListCoins.Sep 27 2019, 16:59
deadalnix edited the summary of this revision. (Show Details)
jasonbcox accepted this revision.Sep 27 2019, 21:59
This revision is now accepted and ready to land.Sep 27 2019, 21:59