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.
This commit is contained in:
Wladimir J. van der Laan 2014-02-18 15:23:24 +01:00
parent c7c3262774
commit 09ec3af166
2 changed files with 2 additions and 3 deletions

View File

@ -470,6 +470,7 @@ bool CWallet::AddToWallet(const CWalletTx& wtxIn, bool fFromLoadWallet)
if (fFromLoadWallet) if (fFromLoadWallet)
{ {
mapWallet[hash] = wtxIn; mapWallet[hash] = wtxIn;
mapWallet[hash].BindWallet(this);
AddToSpends(hash); AddToSpends(hash);
} }
else else

View File

@ -352,9 +352,7 @@ ReadKeyValue(CWallet* pwallet, CDataStream& ssKey, CDataStream& ssValue,
CWalletTx wtx; CWalletTx wtx;
ssValue >> wtx; ssValue >> wtx;
CValidationState state; CValidationState state;
if (CheckTransaction(wtx, state) && (wtx.GetHash() == hash) && state.IsValid()) if (!(CheckTransaction(wtx, state) && (wtx.GetHash() == hash) && state.IsValid()))
wtx.BindWallet(pwallet);
else
return false; return false;
// Undo serialize changes in 31600 // Undo serialize changes in 31600