aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Newbery <john@johnnewbery.com>2021-10-19 12:12:56 +0100
committerJohn Newbery <john@johnnewbery.com>2021-11-03 14:28:04 +0000
commit4c24142b1ec121623f81ba644d77341bc1bd88dd (patch)
tree9e0525a9eadd118dabcf5405ad2d6d7062b6b129
parent5759fd12b8d5937e9187fa33489a95b1d8e6d1e5 (diff)
downloadbitcoin-4c24142b1ec121623f81ba644d77341bc1bd88dd.tar.xz
[validation] Remove comment about AcceptToMemoryPool()
"This logic is not necessary for memory pool transactions, as AcceptToMemoryPool already refuses previously-known transaction ids entirely." refers to the logic at https://github.com/bitcoin/bitcoin/blob/a206b0ea12eb4606b93323268fc81a4f1f952531/src/main.cpp#L484-L486, which was later removed in commit 450cbb0944cd20a06ce806e6679a1f4c83c50db2.
-rw-r--r--src/validation.cpp2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/validation.cpp b/src/validation.cpp
index 207cdc8233..f1c44d61ae 100644
--- a/src/validation.cpp
+++ b/src/validation.cpp
@@ -1658,8 +1658,6 @@ bool CChainState::ConnectBlock(const CBlock& block, BlockValidationState& state,
// can be duplicated to remove the ability to spend the first instance -- even after
// being sent to another address.
// See BIP30, CVE-2012-1909, and http://r6.ca/blog/20120206T005236Z.html for more information.
- // This logic is not necessary for memory pool transactions, as AcceptToMemoryPool
- // already refuses previously-known transaction ids entirely.
// This rule was originally applied to all blocks with a timestamp after March 15, 2012, 0:00 UTC.
// Now that the whole chain is irreversibly beyond that time it is applied to all blocks except the
// two in the chain that violate it. This prevents exploiting the issue against nodes during their