Merge pull request #1182 from blackjok3rtt/FSM
Corrupt tx in wallet fix.
This commit is contained in:
@@ -39,6 +39,7 @@
|
|||||||
using namespace std;
|
using namespace std;
|
||||||
|
|
||||||
static uint64_t nAccountingEntryNumber = 0;
|
static uint64_t nAccountingEntryNumber = 0;
|
||||||
|
static list<uint256> deadTxns;
|
||||||
|
|
||||||
//
|
//
|
||||||
// CWalletDB
|
// CWalletDB
|
||||||
@@ -484,8 +485,11 @@ ReadKeyValue(CWallet* pwallet, CDataStream& ssKey, CDataStream& ssValue,
|
|||||||
CValidationState state;
|
CValidationState state;
|
||||||
auto verifier = libzcash::ProofVerifier::Strict();
|
auto verifier = libzcash::ProofVerifier::Strict();
|
||||||
if (!(CheckTransaction(0,wtx, state, verifier) && (wtx.GetHash() == hash) && state.IsValid()))
|
if (!(CheckTransaction(0,wtx, state, verifier) && (wtx.GetHash() == hash) && state.IsValid()))
|
||||||
|
{
|
||||||
|
fprintf(stderr, "Removing corrupt tx from wallet.%s\n", hash.ToString().c_str());
|
||||||
|
deadTxns.push_back(hash);
|
||||||
return false;
|
return false;
|
||||||
|
}
|
||||||
// Undo serialize changes in 31600
|
// Undo serialize changes in 31600
|
||||||
if (31404 <= wtx.fTimeReceivedIsTxTime && wtx.fTimeReceivedIsTxTime <= 31703)
|
if (31404 <= wtx.fTimeReceivedIsTxTime && wtx.fTimeReceivedIsTxTime <= 31703)
|
||||||
{
|
{
|
||||||
@@ -933,9 +937,6 @@ DBErrors CWalletDB::LoadWallet(CWallet* pwallet)
|
|||||||
{
|
{
|
||||||
// Leave other errors alone, if we try to fix them we might make things worse.
|
// Leave other errors alone, if we try to fix them we might make things worse.
|
||||||
fNoncriticalErrors = true; // ... but do warn the user there is something wrong.
|
fNoncriticalErrors = true; // ... but do warn the user there is something wrong.
|
||||||
if (strType == "tx")
|
|
||||||
// Rescan if there is a bad transaction record:
|
|
||||||
SoftSetBoolArg("-rescan", true);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (!strErr.empty())
|
if (!strErr.empty())
|
||||||
@@ -950,6 +951,24 @@ DBErrors CWalletDB::LoadWallet(CWallet* pwallet)
|
|||||||
result = DB_CORRUPT;
|
result = DB_CORRUPT;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (!deadTxns.empty())
|
||||||
|
{
|
||||||
|
int32_t reAdded = 0;
|
||||||
|
BOOST_FOREACH (uint256& hash, deadTxns) {
|
||||||
|
if (!EraseTx(hash))
|
||||||
|
fprintf(stderr, "could not delete tx.%s\n",hash.ToString().c_str());
|
||||||
|
uint256 blockhash; CTransaction tx;
|
||||||
|
if (GetTransaction(hash,tx,blockhash,true))
|
||||||
|
{
|
||||||
|
CWalletTx wtx(pwallet,tx);
|
||||||
|
pwallet->AddToWallet(wtx, true, NULL);
|
||||||
|
reAdded++;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
fprintf(stderr, "Cleared %lu corrupted transactions from wallet. Readded %i known transactions.\n",deadTxns.size(),reAdded);
|
||||||
|
deadTxns.clear();
|
||||||
|
}
|
||||||
|
|
||||||
if (fNoncriticalErrors && result == DB_LOAD_OK)
|
if (fNoncriticalErrors && result == DB_LOAD_OK)
|
||||||
result = DB_NONCRITICAL_ERROR;
|
result = DB_NONCRITICAL_ERROR;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user