From 3c8a9aefff3a600bef3da63c560f62af9e5582d6 Mon Sep 17 00:00:00 2001 From: Alex Morcos Date: Thu, 8 Jun 2017 12:08:32 -0400 Subject: [PATCH] Add belt-and-suspenders in DisconnectBlock These extra variables were previously checked before the move to per-txout database. --- src/validation.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/validation.cpp b/src/validation.cpp index eb6ea42b6..baf76e5ba 100644 --- a/src/validation.cpp +++ b/src/validation.cpp @@ -1364,6 +1364,7 @@ static DisconnectResult DisconnectBlock(const CBlock& block, const CBlockIndex* for (int i = block.vtx.size() - 1; i >= 0; i--) { const CTransaction &tx = *(block.vtx[i]); uint256 hash = tx.GetHash(); + bool is_coinbase = tx.IsCoinBase(); // Check that all outputs are available and match the outputs in the block itself // exactly. @@ -1372,7 +1373,7 @@ static DisconnectResult DisconnectBlock(const CBlock& block, const CBlockIndex* COutPoint out(hash, o); Coin coin; bool is_spent = view.SpendCoin(out, &coin); - if (!is_spent || tx.vout[o] != coin.out) { + if (!is_spent || tx.vout[o] != coin.out || pindex->nHeight != coin.nHeight || is_coinbase != coin.fCoinBase) { fClean = false; // transaction output mismatch } }