Fix test locking issues and un-revert the probable-deadlines assertions commit #6534

pull theuni wants to merge 2 commits into bitcoin:master from theuni:lock-fix changing 2 files +59 −29
  1. theuni commented at 8:49 PM on August 7, 2015: member

    Fixes #6515.

    The rpc_addmultisig test was failing because cs_wallet was locked prior to rpc functions, upsetting the locking order.

    The rpc_wallet test wasn't failing, but I dropped the lock on cs_wallet for consistency.

  2. locks: don't lock cs_wallet before making rpc calls 605433ffb5
  3. Revert "Revert "Assert on probable deadlocks if the second lock isnt try_lock""
    This reverts commit a4fe57da6207c1e5691a1e843d22db571f3f0186.
    
    The issue here should be fixed by the previous commit.
    9493803f4a
  4. laanwj commented at 3:36 PM on August 10, 2015: member

    ACK

  5. laanwj merged this on Aug 10, 2015
  6. laanwj closed this on Aug 10, 2015

  7. laanwj referenced this in commit 6c1c7fdfd6 on Aug 10, 2015
  8. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me