arno/trezor-firmware
1
0
Fork 0
mirror of https://github.com/trezor/trezor-firmware.git synced 2024-12-18 12:28:09 +00:00
Code Issues Releases Wiki Activity

core/sign_tx: Use varint length encoding for witness stack items.

Browse Source
This commit is contained in:
Andrew Kozlik 2020-05-25 15:39:10 +02:00 committed by Andrew Kozlik
parent 0c734dc8fd
commit e7f230d66e
1 changed files with 15 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
core/src/apps/bitcoin/scripts.py
View File

@ -16,7 +16,12 @@ from .multisig import (
multisig_get_pubkeys,
multisig_pubkey_index,
)
from .writers import write_bytes_fixed, write_bytes_unchecked, write_op_push
from .writers import (
write_bytes_fixed,
write_bytes_prefixed,
write_bytes_unchecked,
write_op_push,
)
if False:
from typing import List, Optional
@ -245,8 +250,8 @@ def input_script_p2wsh_in_p2sh(script_hash: bytes) -> bytearray:
def witness_p2wpkh(signature: bytes, pubkey: bytes, sighash: int) -> bytearray:
w = empty_bytearray(1 + 5 + len(signature) + 1 + 5 + len(pubkey))
write_bitcoin_varint(w, 0x02) # num of segwit items, in P2WPKH it's always 2
append_signature(w, signature, sighash)
append_pubkey(w, pubkey)
write_signature_prefixed(w, signature, sighash)
write_bytes_prefixed(w, pubkey)
return w
@ -290,7 +295,7 @@ def witness_p2wsh(
write_bitcoin_varint(w, 0)
for s in signatures:
append_signature(w, s, sighash) # size of the witness included
write_signature_prefixed(w, s, sighash) # size of the witness included
# redeem script
write_bitcoin_varint(w, redeem_script_length)
@ -389,6 +394,12 @@ def output_script_paytoopreturn(data: bytes) -> bytearray:
# ===
def write_signature_prefixed(w: Writer, signature: bytes, sighash: int) -> None:
write_bitcoin_varint(w, len(signature) + 1)
write_bytes_unchecked(w, signature)
w.append(sighash)
def append_signature(w: Writer, signature: bytes, sighash: int) -> None:
write_op_push(w, len(signature) + 1)
write_bytes_unchecked(w, signature)