diff options
| author | Gavin Andresen <[email protected]> | 2012-04-29 20:56:55 -0400 |
|---|---|---|
| committer | Gavin Andresen <[email protected]> | 2012-04-30 11:08:07 -0400 |
| commit | be8651dde7b59e50e8c443da71c706667803d06d (patch) | |
| tree | 357c7714553fa88b628fb85e88b9627b063e2e54 /src/main.cpp | |
| parent | Merge pull request #1139 from Diapolo/messagepage (diff) | |
| download | discoin-be8651dde7b59e50e8c443da71c706667803d06d.tar.xz discoin-be8651dde7b59e50e8c443da71c706667803d06d.zip | |
Check earlier for blocks with duplicate transactions. Fixes #1167v0.6.1rc2
Diffstat (limited to 'src/main.cpp')
| -rw-r--r-- | src/main.cpp | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/main.cpp b/src/main.cpp index e5d110d8e..427e435a9 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -1652,6 +1652,16 @@ bool CBlock::CheckBlock() const if (!tx.CheckTransaction()) 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; BOOST_FOREACH(const CTransaction& tx, vtx) { |