Merge #9771: Add missing cs_wallet lock that triggers new lock held assertion

laanwj · laanwj · commit e43a58514dd3 · 2017-02-16T10:30:33.000+01:00
07afcd6 Add missing cs_wallet lock that triggers new lock held assertion (Russell Yanofsky)
diff --git a/src/wallet/walletdb.cpp b/src/wallet/walletdb.cpp
@@ -559,8 +559,8 @@ DBErrors CWalletDB::LoadWallet(CWallet* pwallet)
     bool fNoncriticalErrors = false;
     DBErrors result = DB_LOAD_OK;
 
+    LOCK(pwallet->cs_wallet);
     try {
-        LOCK(pwallet->cs_wallet);
         int nMinVersion = 0;
         if (Read((string)"minversion", nMinVersion))
         {

Original file line number	Diff line number	Diff line change
`@@ -559,8 +559,8 @@ DBErrors CWalletDB::LoadWallet(CWallet* pwallet)`
`559`	`559`	`bool fNoncriticalErrors = false;`
`560`	`560`	`DBErrors result = DB_LOAD_OK;`
`561`	`561`
	`562`	`+ LOCK(pwallet->cs_wallet);`
`562`	`563`	`try {`
`563`		`- LOCK(pwallet->cs_wallet);`
`564`	`564`	`int nMinVersion = 0;`
`565`	`565`	`if (Read((string)"minversion", nMinVersion))`
`566`	`566`	`{`