mirror of
https://github.com/bitcoin/bips.git
synced 2025-05-12 12:03:29 +00:00
BIP 341: Fix taproot_tweak_pubkey
`lift_x` returns `None` if the input integer is not an X coordinate on the curve to indicate failure. `point_add`, on the other hand, interprets `None` as the point at infinity. Therefore, without this commit, if the internal `pubkey` is not a valid X coordinate, the function will not fail, which contradicts the specification in the "Script validation rules section". Instead, it sets `Q` to `t*G`.
This commit is contained in:
parent
6545b81022
commit
be340277fc
@ -182,7 +182,10 @@ def taproot_tweak_pubkey(pubkey, h):
|
||||
t = int_from_bytes(tagged_hash("TapTweak", pubkey + h))
|
||||
if t >= SECP256K1_ORDER:
|
||||
raise ValueError
|
||||
Q = point_add(lift_x(int(pubkey)), point_mul(G, t))
|
||||
P = lift_x(int_from_bytes(pubkey))
|
||||
if P is None:
|
||||
raise ValueError
|
||||
Q = point_add(P, point_mul(G, t))
|
||||
return 0 if has_even_y(Q) else 1, bytes_from_int(x(Q))
|
||||
|
||||
def taproot_tweak_seckey(seckey0, h):
|
||||
|
Loading…
x
Reference in New Issue
Block a user