diff options
author | fanquake <fanquake@gmail.com> | 2019-08-30 09:10:08 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2019-08-30 09:22:31 +0800 |
commit | a8ecd0dadaabaaf8c87dfc42d5e27cf2cd714132 (patch) | |
tree | 6220e74c9bd8aae07259e79edae7a3faa013d346 /ci/test/06_script_b.sh | |
parent | 74da99e010744e8759e728a6e136e13c9b3ba433 (diff) | |
parent | 798a589aff64b83a0844688a661f4bd987c3340c (diff) |
Merge #16753: wallet: extract PubKey from P2PK script with Solver
798a589aff64b83a0844688a661f4bd987c3340c wallet: extract PubKey from P2PK script with Solver (Sebastian Falbesoner)
Pull request description:
The function `ExtractPubKey()` checks if a given script matches the P2PK pattern
(`<PubKey> OP_CHECKSIG`), extracts the PubKey and additionally checks if it is
cryptographically valid (full validation with ECC library via `CPubKey::IsFullyValid()`).
Currently this is done manually in the following order:
1. check if first script OP is data push with valid PubKey length (first part of pattern match), extract PubKey
2. create `CPubKey` object with extracted PubKey
3. fully validate public key
4. check if last script OP is `OP_CHECKSIG` (second part of pattern match)
Using Solver, the pattern matching and PubKey extraction can be done via a
single step, leading to the following simplified order with shorter code:
1. check if given script matches P2PK pattern with Solver (also contains valid PubKey length check), extracts Pubkey
2. create `CPubKey` object with extracted Pubkey
3. fully validate public key
ACKs for top commit:
instagibbs:
utACK https://github.com/bitcoin/bitcoin/pull/16753/commits/798a589aff64b83a0844688a661f4bd987c3340c
theStack:
> utACK [798a589](https://github.com/bitcoin/bitcoin/commit/798a589aff64b83a0844688a661f4bd987c3340c)
sipa:
ACK 798a589aff64b83a0844688a661f4bd987c3340c
achow101:
Code Review ACK 798a589aff64b83a0844688a661f4bd987c3340c
Tree-SHA512: 350358a89afed8c2a7967c50e9714a2d4a909259b50e694ce68dde3e7d0fa0bf3238d33642e73f2bdb53860f6d3f7327ca3eb6426b74eaffacfbca0a384d68cd
Diffstat (limited to 'ci/test/06_script_b.sh')
0 files changed, 0 insertions, 0 deletions