aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWladimir J. van der Laan <laanwj@gmail.com>2014-02-18 15:23:24 +0100
committerWladimir J. van der Laan <laanwj@gmail.com>2014-03-07 09:20:11 +0100
commit09ec3af16687aa301a921a1a6e94d7a57d4af141 (patch)
tree7fea9f9994827e4b6156f59484d13eef906d6708
parentc7c32627742575afeadca636df09e7014135e27b (diff)
AddToWallet implies BindWallet
Now that AddToWallet is called when loading transactions from the wallet database, BindWallet can be integrated into that and does not need to be an extra step. Leaves behaviour unchanged, but makes the fFromLoadWallet/!fFromLoadWallet paths in AddToWallet a bit more symmetric.
-rw-r--r--src/wallet.cpp1
-rw-r--r--src/walletdb.cpp4
2 files changed, 2 insertions, 3 deletions
diff --git a/src/wallet.cpp b/src/wallet.cpp
index 3ecd994e9d..352e2a776e 100644
--- a/src/wallet.cpp
+++ b/src/wallet.cpp
@@ -470,6 +470,7 @@ bool CWallet::AddToWallet(const CWalletTx& wtxIn, bool fFromLoadWallet)
if (fFromLoadWallet)
{
mapWallet[hash] = wtxIn;
+ mapWallet[hash].BindWallet(this);
AddToSpends(hash);
}
else
diff --git a/src/walletdb.cpp b/src/walletdb.cpp
index b57ea0b518..359a1cef61 100644
--- a/src/walletdb.cpp
+++ b/src/walletdb.cpp
@@ -352,9 +352,7 @@ ReadKeyValue(CWallet* pwallet, CDataStream& ssKey, CDataStream& ssValue,
CWalletTx wtx;
ssValue >> wtx;
CValidationState state;
- if (CheckTransaction(wtx, state) && (wtx.GetHash() == hash) && state.IsValid())
- wtx.BindWallet(pwallet);
- else
+ if (!(CheckTransaction(wtx, state) && (wtx.GetHash() == hash) && state.IsValid()))
return false;
// Undo serialize changes in 31600