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 /test/functional/data | |
parent | 8cca1fbea9bc7bcd32600f3ba5e81cfefb4309af (diff) | |
parent | bb530efa1872ec963417f61da9a95185c7a7a7d6 (diff) | |
download | bitcoin-c65c77c721b69f1782509145d62978d31c9a1d78.tar.xz |
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 'test/functional/data')
0 files changed, 0 insertions, 0 deletions