Split up CheckBlock in a block and header version
This commit is contained in:
parent
4765b8c116
commit
f457347053
2 changed files with 18 additions and 9 deletions
26
src/main.cpp
26
src/main.cpp
|
@ -2288,16 +2288,8 @@ bool FindUndoPos(CValidationState &state, int nFile, CDiskBlockPos &pos, unsigne
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
bool CheckBlock(const CBlock& block, CValidationState& state, bool fCheckPOW, bool fCheckMerkleRoot)
|
bool CheckBlockHeader(const CBlockHeader& block, CValidationState& state, bool fCheckPOW)
|
||||||
{
|
{
|
||||||
// These are checks that are independent of context
|
|
||||||
// that can be verified before saving an orphan block.
|
|
||||||
|
|
||||||
// Size limits
|
|
||||||
if (block.vtx.empty() || block.vtx.size() > MAX_BLOCK_SIZE || ::GetSerializeSize(block, SER_NETWORK, PROTOCOL_VERSION) > MAX_BLOCK_SIZE)
|
|
||||||
return state.DoS(100, error("CheckBlock() : size limits failed"),
|
|
||||||
REJECT_INVALID, "bad-blk-length");
|
|
||||||
|
|
||||||
// Check proof of work matches claimed amount
|
// Check proof of work matches claimed amount
|
||||||
if (fCheckPOW && !CheckProofOfWork(block.GetHash(), block.nBits))
|
if (fCheckPOW && !CheckProofOfWork(block.GetHash(), block.nBits))
|
||||||
return state.DoS(50, error("CheckBlock() : proof of work failed"),
|
return state.DoS(50, error("CheckBlock() : proof of work failed"),
|
||||||
|
@ -2308,6 +2300,22 @@ bool CheckBlock(const CBlock& block, CValidationState& state, bool fCheckPOW, bo
|
||||||
return state.Invalid(error("CheckBlock() : block timestamp too far in the future"),
|
return state.Invalid(error("CheckBlock() : block timestamp too far in the future"),
|
||||||
REJECT_INVALID, "time-too-new");
|
REJECT_INVALID, "time-too-new");
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
bool CheckBlock(const CBlock& block, CValidationState& state, bool fCheckPOW, bool fCheckMerkleRoot)
|
||||||
|
{
|
||||||
|
// These are checks that are independent of context
|
||||||
|
// that can be verified before saving an orphan block.
|
||||||
|
|
||||||
|
if (!CheckBlockHeader(block, state, fCheckPOW))
|
||||||
|
return false;
|
||||||
|
|
||||||
|
// Size limits
|
||||||
|
if (block.vtx.empty() || block.vtx.size() > MAX_BLOCK_SIZE || ::GetSerializeSize(block, SER_NETWORK, PROTOCOL_VERSION) > MAX_BLOCK_SIZE)
|
||||||
|
return state.DoS(100, error("CheckBlock() : size limits failed"),
|
||||||
|
REJECT_INVALID, "bad-blk-length");
|
||||||
|
|
||||||
// First transaction must be coinbase, the rest must not be
|
// First transaction must be coinbase, the rest must not be
|
||||||
if (block.vtx.empty() || !block.vtx[0].IsCoinBase())
|
if (block.vtx.empty() || !block.vtx[0].IsCoinBase())
|
||||||
return state.DoS(100, error("CheckBlock() : first tx is not coinbase"),
|
return state.DoS(100, error("CheckBlock() : first tx is not coinbase"),
|
||||||
|
|
|
@ -603,6 +603,7 @@ bool ConnectBlock(CBlock& block, CValidationState& state, CBlockIndex* pindex, C
|
||||||
bool AddToBlockIndex(CBlock& block, CValidationState& state, const CDiskBlockPos& pos);
|
bool AddToBlockIndex(CBlock& block, CValidationState& state, const CDiskBlockPos& pos);
|
||||||
|
|
||||||
// Context-independent validity checks
|
// Context-independent validity checks
|
||||||
|
bool CheckBlockHeader(const CBlockHeader& block, CValidationState& state, bool fCheckPOW = true);
|
||||||
bool CheckBlock(const CBlock& block, CValidationState& state, bool fCheckPOW = true, bool fCheckMerkleRoot = true);
|
bool CheckBlock(const CBlock& block, CValidationState& state, bool fCheckPOW = true, bool fCheckMerkleRoot = true);
|
||||||
|
|
||||||
// Store block on disk
|
// Store block on disk
|
||||||
|
|
Loading…
Reference in a new issue