diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2020-04-07 00:59:40 +0800 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2020-04-07 00:59:48 +0800 |
| commit | c0b389b33516fb3eaaad7c30bd11dba768882a7e (patch) | |
| tree | d8d72b8435b3b16a4703bfdc27af34bc38504886 /test/functional/p2p_filter.py | |
| parent | 425a7f900ed824caa5d4a93991542c70171539fe (diff) | |
| parent | cd3b1569d9ad8e24d3a222aff74e0c254baadf79 (diff) | |
| download | bitcoin-c0b389b33516fb3eaaad7c30bd11dba768882a7e.tar.xz | |
Merge #18484: rpc: Correctly compute redeemScript from witnessScript for signrawtransaction
cd3b1569d9ad8e24d3a222aff74e0c254baadf79 Correctly compute redeemScript from witnessScript for signrawtransaction (Andrew Chow)
Pull request description:
`ParsePrevouts` uses `GetScriptForWitness` on the given witnessScript to find the corresponding redeemScript. This is incorrect when the witnessScript is either a P2PK or P2PKH script as it returns the corresponding P2WPK script instead of turning the witnessScript into a P2WSH script. Instead this should make the script a `WitnessV0ScriptHash` destination and get the script for that.
Test cases are also added. These will fail on master with a `redeemScript does not correspond to witnessScript`
Reported on [Bitcointalk](https://bitcointalk.org/index.php?topic=5236818.0)
ACKs for top commit:
MarcoFalke:
weak ACK cd3b1569d9, only checked that the test fails without the code change 🚰
instagibbs:
utACK https://github.com/bitcoin/bitcoin/pull/18484/commits/cd3b1569d9ad8e24d3a222aff74e0c254baadf79
Tree-SHA512: afac671dbb52ce88bfb4a9ca3dd6065427ad52c9778d0549ad40e9286778f308adad24fb3b3c3089545d7f88c57c53d41224fd7a4bb207550eff2fe06600118f
Diffstat (limited to 'test/functional/p2p_filter.py')
0 files changed, 0 insertions, 0 deletions