Check earlier for blocks with duplicate transactions. Fixes #1167
This commit is contained in:
parent
ec4997d48f
commit
be8651dde7
10
src/main.cpp
10
src/main.cpp
@ -1652,6 +1652,16 @@ bool CBlock::CheckBlock() const
|
|||||||
if (!tx.CheckTransaction())
|
if (!tx.CheckTransaction())
|
||||||
return DoS(tx.nDoS, error("CheckBlock() : CheckTransaction failed"));
|
return DoS(tx.nDoS, error("CheckBlock() : CheckTransaction failed"));
|
||||||
|
|
||||||
|
// Check for duplicate txids. This is caught by ConnectInputs(),
|
||||||
|
// but catching it earlier avoids a potential DoS attack:
|
||||||
|
set<uint256> uniqueTx;
|
||||||
|
BOOST_FOREACH(const CTransaction& tx, vtx)
|
||||||
|
{
|
||||||
|
uniqueTx.insert(tx.GetHash());
|
||||||
|
}
|
||||||
|
if (uniqueTx.size() != vtx.size())
|
||||||
|
return DoS(100, error("CheckBlock() : duplicate transaction"));
|
||||||
|
|
||||||
unsigned int nSigOps = 0;
|
unsigned int nSigOps = 0;
|
||||||
BOOST_FOREACH(const CTransaction& tx, vtx)
|
BOOST_FOREACH(const CTransaction& tx, vtx)
|
||||||
{
|
{
|
||||||
|
Loading…
Reference in New Issue
Block a user