diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2019-04-25 14:15:40 -0400 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2019-04-25 14:15:43 -0400 |
| commit | c65c77c721b69f1782509145d62978d31c9a1d78 (patch) | |
| tree | c63bc16ca5022365ab1df8fb7d540a2f01983abc /build_msvc/bitcoin_config.h | |
| parent | 8cca1fbea9bc7bcd32600f3ba5e81cfefb4309af (diff) | |
| parent | bb530efa1872ec963417f61da9a95185c7a7a7d6 (diff) | |
Merge #14039: Disallow extended encoding for non-witness transactions
bb530efa18 Disallow extended encoding for non-witness transactions (Pieter Wuille)
Pull request description:
BIP144 specifies that transactions without witness should use the legacy encoding, which is currently not enforced.
This rule was present in the original SegWit implementation (https://github.com/bitcoin/bitcoin/pull/8149), but was subsequently dropped (https://github.com/bitcoin/bitcoin/pull/8589).
As all hashes, txids, and weights are always computed over a reserialized version of a transaction, it is mostly harmless to permit extended encoding for non-segwit transactions, but I'd rather strictly follow the BIP.
ACKs for commit bb530e:
instagibbs:
utACK https://github.com/bitcoin/bitcoin/pull/14039/commits/bb530efa1872ec963417f61da9a95185c7a7a7d6
stevenroose:
utACK bb530efa1872ec963417f61da9a95185c7a7a7d6
Tree-SHA512: 1aeccd6a555f43784fefb076ce2e8ad2f5ba7be49840544a50050d0390f82373f87201bf56cf8bb30841b4f9cd893b382261a080da875d4e11ab7051f8640dbe
Diffstat (limited to 'build_msvc/bitcoin_config.h')
0 files changed, 0 insertions, 0 deletions