Browse Source

Avoid coredumps in ReacceptWalletTransactions and log the situation, which is possibly caused by corrupt wallet tx data

pull/195/head
Duke Leto 2 years ago
parent
commit
bd9006623c
  1. 7
      src/wallet/wallet.cpp

7
src/wallet/wallet.cpp

@ -2847,7 +2847,12 @@ void CWallet::ReacceptWalletTransactions()
{
const uint256& wtxid = item.first;
CWalletTx& wtx = item.second;
assert(wtx.GetHash() == wtxid);
if(wtx.GetHash() != wtxid) {
LogPrintf("%s: Something funky going on, skipping this tx. wtx.GetHash() != wtxid (%s != %s)\n", __func__, wtx.GetHash().ToString().c_str(), wtxid.ToString().c_str() );
continue;
}
// Crashing the node because of this is lame
// assert(wtx.GetHash() == wtxid);
int nDepth = wtx.GetDepthInMainChain();

Loading…
Cancel
Save