diff options
author | Cory Fields <cory-nospam-@coryfields.com> | 2015-06-16 04:02:25 -0400 |
---|---|---|
committer | Luke Dashjr <luke-jr+git@utopios.org> | 2015-09-22 00:43:10 +0000 |
commit | 60457d3c2f3f38d4cc3e524a5d4030bf4034b7aa (patch) | |
tree | f78d5f3a5ff4255548f103db72d242e50da8dd6b /src/wallet/rpcwallet.cpp | |
parent | a496e11d7cf827d69c309faf0da9a724d98916c5 (diff) | |
download | bitcoin-60457d3c2f3f38d4cc3e524a5d4030bf4034b7aa.tar.xz |
locking: fix a few small issues uncovered by -Wthread-safety
- rpcwallet: No need to lock twice here
- openssl: Clang doesn't understand selective lock/unlock here. Ignore it.
- CNode: Fix a legitimate (though very unlikely) locking bug.
Diffstat (limited to 'src/wallet/rpcwallet.cpp')
-rw-r--r-- | src/wallet/rpcwallet.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/wallet/rpcwallet.cpp b/src/wallet/rpcwallet.cpp index 7a0e5989e0..bfaaaeb4c8 100644 --- a/src/wallet/rpcwallet.cpp +++ b/src/wallet/rpcwallet.cpp @@ -476,7 +476,6 @@ Value listaddressgroupings(const Array& params, bool fHelp) addressInfo.push_back(CBitcoinAddress(address).ToString()); addressInfo.push_back(ValueFromAmount(balances[address])); { - LOCK(pwalletMain->cs_wallet); if (pwalletMain->mapAddressBook.find(CBitcoinAddress(address).Get()) != pwalletMain->mapAddressBook.end()) addressInfo.push_back(pwalletMain->mapAddressBook.find(CBitcoinAddress(address).Get())->second.name); } |