diff options
| author | Johnson Lau <jl2012@xbt.hk> | 2018-04-27 03:31:36 +0800 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2018-05-24 14:03:05 -0400 |
| commit | 0a000b9b73a7d7bb4086aeefc86841d0fc33e652 (patch) | |
| tree | 6568d7a99f18892716c3befdef3892af3c5fbded /src | |
| parent | 1fffc2b346b2d2e129db5c9f5cad00e820c85c45 (diff) | |
| download | bitcoin-0a000b9b73a7d7bb4086aeefc86841d0fc33e652.tar.xz | |
Policy to reject extremely small transactions
A transaction with 1 segwit input and 1 P2WPHK output has non-witness size of 82 bytes. Anything smaller than this have unnecessary malloc overhead and are not relayed/mined.
Github-Pull: #11423
Rebased-From: 7485488e907e236133a016ba7064c89bf9ab6da3
Diffstat (limited to 'src')
| -rw-r--r-- | src/policy/policy.h | 2 | ||||
| -rw-r--r-- | src/validation.cpp | 6 |
2 files changed, 8 insertions, 0 deletions
diff --git a/src/policy/policy.h b/src/policy/policy.h index 88dae4ab74..be1c0e03ec 100644 --- a/src/policy/policy.h +++ b/src/policy/policy.h @@ -22,6 +22,8 @@ static const unsigned int DEFAULT_BLOCK_MAX_WEIGHT = MAX_BLOCK_WEIGHT - 4000; static const unsigned int DEFAULT_BLOCK_MIN_TX_FEE = 1000; /** The maximum weight for transactions we're willing to relay/mine */ static const unsigned int MAX_STANDARD_TX_WEIGHT = 400000; +/** The minimum non-witness size for transactions we're willing to relay/mine (1 segwit input + 1 P2WPKH output = 82 bytes) */ +static const unsigned int MIN_STANDARD_TX_NONWITNESS_SIZE = 82; /** Maximum number of signature check operations in an IsStandard() P2SH script */ static const unsigned int MAX_P2SH_SIGOPS = 15; /** The maximum number of sigops we're willing to relay/mine in a single tx */ diff --git a/src/validation.cpp b/src/validation.cpp index 7c4c84c3f9..2c5b5a1a31 100644 --- a/src/validation.cpp +++ b/src/validation.cpp @@ -576,6 +576,12 @@ static bool AcceptToMemoryPoolWorker(const CChainParams& chainparams, CTxMemPool if (fRequireStandard && !IsStandardTx(tx, reason, witnessEnabled)) return state.DoS(0, false, REJECT_NONSTANDARD, reason); + // Do not work on transactions that are too small. + // A transaction with 1 segwit input and 1 P2WPHK output has non-witness size of 82 bytes. + // Transactions smaller than this are not relayed to reduce unnecessary malloc overhead. + if (::GetSerializeSize(tx, SER_NETWORK, PROTOCOL_VERSION | SERIALIZE_TRANSACTION_NO_WITNESS) < MIN_STANDARD_TX_NONWITNESS_SIZE) + return state.DoS(0, false, REJECT_NONSTANDARD, "tx-size-small"); + // Only accept nLockTime-using transactions that can be mined in the next // block; we don't want our mempool filled up with transactions that can't // be mined yet. |