mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 12:32:48 +01:00
be5c84f41d
d6d2ab984547be4a9f7ba859a2a4c9ac9bfbf206 test: MiniWallet: fix fee calculation for P2PK and check tx vsize (Sebastian Falbesoner) ce024b1c0ef2dcd307023aaaab40373c8bf17db1 test: MiniWallet: force P2PK signature to have fixed size (71 bytes) (Sebastian Falbesoner) Pull request description: This PR is a follow-up to #21945. It aims to both fix the fee calculation for P2PK mode transactions and enable its vsize check. Currently, the latter assumes a fixed tx length, which is fine for anyone-can-spend txs but doesn't apply to P2PK output spends due to varying DER signature size; the vsize check is therefore disabled for P2PK mode on master branch. Creating one million DER signatures with MiniWallet shows the following distribution of sizes (smart people with better math skills probably could deduce the ratios without trying, but hey): | DER signature size [bytes] | #occurences (ratio) | | ------------- | ------------- | | 71 | 498893 (49.89%) | | 70 | 497244 (49.72%) | | 69 | 3837 (0.38%) | | 68 | 22 (0.0022%) | Note that even smaller signatures are possible (for smaller R and S values with leading zero bytes), it's just that the probability decreases exponentially. Instead of choosing a large vsize check range and hoping that smaller signatures are never created (potentially leading to flaky tests), the proposed solution is ~~to limit the signature size to the two most common sizes 71 and 70 (>99.6% probability) and then accordingly only check for two vsize values; the value to be used for fee calculation is a decimal right between the two possible sizes (167.5 vbytes) and for the vsize check it's rounded down/up integer values are used.~~ to simply grind the signature to a fixed size of 71 bytes (49.89% probability, i.e. on average each call to `sign_tx()`, on average two ECC signing operations are needed). ~~The idea of grinding signatures to a fixed size (similar to https://github.com/bitcoin/bitcoin/pull/13666 which grinds to low-R values) would be counter-productive, as the signature creation in the test suite is quite expensive and this would significantly slow down tests that calculate hundreds of signatures (like e.g. feature_csv_activation.py).~~ For more about transaction sizes on different input/output types, see the following interesting article: https://medium.com/coinmonks/on-bitcoin-transaction-sizes-97e31bc9d816 ACKs for top commit: MarcoFalke: Concept ACK d6d2ab984547be4a9f7ba859a2a4c9ac9bfbf206 Tree-SHA512: 011c70ee0e4adf9ba12902e4b6c411db9ae96bdd8bc810bf1d67713367998e28ea328394503371fc1f5087a819547ddaea56c073b28db893ae1c0031d7927f32
178 lines
7.7 KiB
Python
178 lines
7.7 KiB
Python
#!/usr/bin/env python3
|
|
# Copyright (c) 2020 The Bitcoin Core developers
|
|
# Distributed under the MIT software license, see the accompanying
|
|
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
|
|
"""A limited-functionality wallet, which may replace a real wallet in tests"""
|
|
|
|
from decimal import Decimal
|
|
from enum import Enum
|
|
from test_framework.address import ADDRESS_BCRT1_P2SH_OP_TRUE
|
|
from test_framework.key import ECKey
|
|
from typing import Optional
|
|
from test_framework.messages import (
|
|
COIN,
|
|
COutPoint,
|
|
CTransaction,
|
|
CTxIn,
|
|
CTxOut,
|
|
)
|
|
from test_framework.script import (
|
|
CScript,
|
|
SignatureHash,
|
|
OP_CHECKSIG,
|
|
OP_TRUE,
|
|
OP_NOP,
|
|
SIGHASH_ALL,
|
|
)
|
|
from test_framework.util import (
|
|
assert_equal,
|
|
hex_str_to_bytes,
|
|
satoshi_round,
|
|
)
|
|
|
|
|
|
class MiniWalletMode(Enum):
|
|
"""Determines the transaction type the MiniWallet is creating and spending.
|
|
|
|
For most purposes, the default mode ADDRESS_OP_TRUE should be sufficient;
|
|
it simply uses a fixed P2SH address whose coins are spent with a
|
|
witness stack of OP_TRUE, i.e. following an anyone-can-spend policy.
|
|
However, if the transactions need to be modified by the user (e.g. prepending
|
|
scriptSig for testing opcodes that are activated by a soft-fork), or the txs
|
|
should contain an actual signature, the raw modes RAW_OP_TRUE and RAW_P2PK
|
|
can be useful. Summary of modes:
|
|
|
|
| output | | tx is | can modify | needs
|
|
mode | description | address | standard | scriptSig | signing
|
|
----------------+-------------------+-----------+----------+------------+----------
|
|
ADDRESS_OP_TRUE | anyone-can-spend | bech32 | yes | no | no
|
|
RAW_OP_TRUE | anyone-can-spend | - (raw) | no | yes | no
|
|
RAW_P2PK | pay-to-public-key | - (raw) | yes | yes | yes
|
|
"""
|
|
ADDRESS_OP_TRUE = 1
|
|
RAW_OP_TRUE = 2
|
|
RAW_P2PK = 3
|
|
|
|
|
|
class MiniWallet:
|
|
def __init__(self, test_node, *, mode=MiniWalletMode.ADDRESS_OP_TRUE):
|
|
self._test_node = test_node
|
|
self._utxos = []
|
|
self._priv_key = None
|
|
self._address = None
|
|
|
|
assert isinstance(mode, MiniWalletMode)
|
|
if mode == MiniWalletMode.RAW_OP_TRUE:
|
|
self._scriptPubKey = bytes(CScript([OP_TRUE]))
|
|
elif mode == MiniWalletMode.RAW_P2PK:
|
|
# use simple deterministic private key (k=1)
|
|
self._priv_key = ECKey()
|
|
self._priv_key.set((1).to_bytes(32, 'big'), True)
|
|
pub_key = self._priv_key.get_pubkey()
|
|
self._scriptPubKey = bytes(CScript([pub_key.get_bytes(), OP_CHECKSIG]))
|
|
elif mode == MiniWalletMode.ADDRESS_OP_TRUE:
|
|
self._address = ADDRESS_BCRT1_P2SH_OP_TRUE
|
|
self._scriptPubKey = hex_str_to_bytes(self._test_node.validateaddress(self._address)['scriptPubKey'])
|
|
|
|
def scan_blocks(self, *, start=1, num):
|
|
"""Scan the blocks for self._address outputs and add them to self._utxos"""
|
|
for i in range(start, start + num):
|
|
block = self._test_node.getblock(blockhash=self._test_node.getblockhash(i), verbosity=2)
|
|
for tx in block['tx']:
|
|
self.scan_tx(tx)
|
|
|
|
def scan_tx(self, tx):
|
|
"""Scan the tx for self._scriptPubKey outputs and add them to self._utxos"""
|
|
for out in tx['vout']:
|
|
if out['scriptPubKey']['hex'] == self._scriptPubKey.hex():
|
|
self._utxos.append({'txid': tx['txid'], 'vout': out['n'], 'value': out['value']})
|
|
|
|
def sign_tx(self, tx, fixed_length=True):
|
|
"""Sign tx that has been created by MiniWallet in P2PK mode"""
|
|
assert self._priv_key is not None
|
|
(sighash, err) = SignatureHash(CScript(self._scriptPubKey), tx, 0, SIGHASH_ALL)
|
|
assert err is None
|
|
# for exact fee calculation, create only signatures with fixed size by default (>49.89% probability):
|
|
# 65 bytes: high-R val (33 bytes) + low-S val (32 bytes)
|
|
# with the DER header/skeleton data of 6 bytes added, this leads to a target size of 71 bytes
|
|
der_sig = b''
|
|
while not len(der_sig) == 71:
|
|
der_sig = self._priv_key.sign_ecdsa(sighash)
|
|
if not fixed_length:
|
|
break
|
|
tx.vin[0].scriptSig = CScript([der_sig + bytes(bytearray([SIGHASH_ALL]))])
|
|
|
|
def generate(self, num_blocks):
|
|
"""Generate blocks with coinbase outputs to the internal address, and append the outputs to the internal list"""
|
|
blocks = self._test_node.generatetodescriptor(num_blocks, f'raw({self._scriptPubKey.hex()})')
|
|
for b in blocks:
|
|
cb_tx = self._test_node.getblock(blockhash=b, verbosity=2)['tx'][0]
|
|
self._utxos.append({'txid': cb_tx['txid'], 'vout': 0, 'value': cb_tx['vout'][0]['value']})
|
|
return blocks
|
|
|
|
def get_address(self):
|
|
return self._address
|
|
|
|
def get_utxo(self, *, txid: Optional[str]='', mark_as_spent=True):
|
|
"""
|
|
Returns a utxo and marks it as spent (pops it from the internal list)
|
|
|
|
Args:
|
|
txid: get the first utxo we find from a specific transaction
|
|
|
|
Note: Can be used to get the change output immediately after a send_self_transfer
|
|
"""
|
|
index = -1 # by default the last utxo
|
|
if txid:
|
|
utxo = next(filter(lambda utxo: txid == utxo['txid'], self._utxos))
|
|
index = self._utxos.index(utxo)
|
|
if mark_as_spent:
|
|
return self._utxos.pop(index)
|
|
else:
|
|
return self._utxos[index]
|
|
|
|
def send_self_transfer(self, **kwargs):
|
|
"""Create and send a tx with the specified fee_rate. Fee may be exact or at most one satoshi higher than needed."""
|
|
tx = self.create_self_transfer(**kwargs)
|
|
self.sendrawtransaction(from_node=kwargs['from_node'], tx_hex=tx['hex'])
|
|
return tx
|
|
|
|
def create_self_transfer(self, *, fee_rate=Decimal("0.003"), from_node, utxo_to_spend=None, mempool_valid=True, locktime=0, sequence=0):
|
|
"""Create and return a tx with the specified fee_rate. Fee may be exact or at most one satoshi higher than needed."""
|
|
self._utxos = sorted(self._utxos, key=lambda k: k['value'])
|
|
utxo_to_spend = utxo_to_spend or self._utxos.pop() # Pick the largest utxo (if none provided) and hope it covers the fee
|
|
if self._priv_key is None:
|
|
vsize = Decimal(85) # anyone-can-spend
|
|
else:
|
|
vsize = Decimal(168) # P2PK (73 bytes scriptSig + 35 bytes scriptPubKey + 60 bytes other)
|
|
send_value = satoshi_round(utxo_to_spend['value'] - fee_rate * (vsize / 1000))
|
|
fee = utxo_to_spend['value'] - send_value
|
|
assert send_value > 0
|
|
|
|
tx = CTransaction()
|
|
tx.vin = [CTxIn(COutPoint(int(utxo_to_spend['txid'], 16), utxo_to_spend['vout']), nSequence=sequence)]
|
|
tx.vout = [CTxOut(int(send_value * COIN), self._scriptPubKey)]
|
|
tx.nLockTime = locktime
|
|
if not self._address:
|
|
# raw script
|
|
if self._priv_key is not None:
|
|
# P2PK, need to sign
|
|
self.sign_tx(tx)
|
|
else:
|
|
# anyone-can-spend
|
|
tx.vin[0].scriptSig = CScript([OP_NOP] * 24) # pad to identical size
|
|
else:
|
|
tx.vin[0].scriptSig = CScript([CScript([OP_TRUE])])
|
|
tx_hex = tx.serialize().hex()
|
|
|
|
tx_info = from_node.testmempoolaccept([tx_hex])[0]
|
|
assert_equal(mempool_valid, tx_info['allowed'])
|
|
if mempool_valid:
|
|
assert_equal(len(tx_hex) // 2, vsize) # 1 byte = 2 character
|
|
assert_equal(tx_info['fees']['base'], fee)
|
|
return {'txid': tx_info['txid'], 'hex': tx_hex, 'tx': tx}
|
|
|
|
def sendrawtransaction(self, *, from_node, tx_hex):
|
|
from_node.sendrawtransaction(tx_hex)
|
|
self.scan_tx(from_node.decoderawtransaction(tx_hex))
|