From 29f29626fc9cbfccf02512a5b31ba283b9d376b8 Mon Sep 17 00:00:00 2001 From: Jochen Hoenicke Date: Sat, 29 Jul 2017 22:03:30 +0200 Subject: [PATCH] Added multisig test --- tests/device_tests/test_msg_signtx_bch.py | 139 +++++++++++++++++++++- 1 file changed, 138 insertions(+), 1 deletion(-) diff --git a/tests/device_tests/test_msg_signtx_bch.py b/tests/device_tests/test_msg_signtx_bch.py index 6c445c79e2..5f5959400a 100644 --- a/tests/device_tests/test_msg_signtx_bch.py +++ b/tests/device_tests/test_msg_signtx_bch.py @@ -73,7 +73,6 @@ class TestMsgSigntxSegwit(common.TrezorTest): ]) (signatures, serialized_tx) = self.client.sign_tx('Bcash Testnet', [inp1, inp2], [out1, out2]) - print(binascii.hexlify(serialized_tx)) self.assertEqual(binascii.hexlify(serialized_tx), b'0100000002243e15b53cc553d93ec4e27e16984adc3d885ef107c613a7577fea47f5dadcd2010000006b483045022100ebcce894cac5d1750f4b1abc7d8c0a5d25944c12a02942b0cc1c89c397acc09602207335077bd698cefc1694d5817abdb3b7aecfcd110ca07729d893577ada71d35441210364430c9122948e525e2f1c6d88f00f47679274f0810fd8c63754954f310995c1ffffffffb02b727cc0c0823f2ac7deb3deb755c8c796e2a777cec5ff727be27d07bc26fe010000006b4830450221008a140434b3f105686c6c4a704c9f2d09b2faf633b81b9ec2304b81f2fd617a9c0220497279b656c4034fd30c41c6f1d0e2ad2aa1b867575aa8f4c13218cb8c114140412103749e3f0a7b01f73427ed67c1cedbb4ecd2315ad6b7c2513355393b95d1ba6137ffffffff0200e1f505000000001976a914b3cc67f3349974d0f1b50e9bb5dfdf226f888fa088acc84ff803000000001976a914f80fb232a1e54b1fa732bc120cae72eabd7fcf6888ac00000000') def test_attack_amount(self): @@ -204,3 +203,141 @@ class TestMsgSigntxSegwit(common.TrezorTest): proto.Failure(code=proto_types.Failure_ProcessError), ]) self.assertRaises(CallException, self.client.sign_tx, 'Bcash Testnet', [inp1, inp2], [out1, out2], debug_processor=attack_processor) + + def test_send_bcc_multisig_wrongchange(self): + self.setup_mnemonic_allallall() + self.client.set_tx_api(TxApiTestnet) + xpubs = []; + for n in map(lambda index : self.client.get_public_node(self.client.expand_path("44'/1'/"+str(index)+"'")), range(1,4)): + xpubs.append(n.xpub) + + def getmultisig(chain, nr, signatures=[b'',b'',b''], xpubs=xpubs): + return proto_types.MultisigRedeemScriptType( + pubkeys=map(lambda xpub : proto_types.HDNodePathType(node=deserialize(xpub), address_n=[chain,nr]), xpubs), + signatures=signatures, + m=2, + ) + correcthorse = proto_types.HDNodeType( + depth=1,fingerprint=0,child_num=0, + chain_code=binascii.unhexlify('0000000000000000000000000000000000000000000000000000000000000000'), + public_key=binascii.unhexlify('0378d430274f8c5ec1321338151e9f27f4c676a008bdf8638d07c0b6be9ab35c71')) + inp1 = proto_types.TxInputType( + address_n=self.client.expand_path("44'/1'/1'/0/0"), + multisig=getmultisig(0,0), + # moUJnmge8SRXuediK7bW6t4YfrPqbE6hD7 + amount=50000000, + prev_hash=binascii.unhexlify('5473e99f750a44de54c6bdaf2de393768e386b34545d2497f9cc6f28d2440289'), + prev_index=0, + script_type=proto_types.SPENDMULTISIG, + ) + out1 = proto_types.TxOutputType( + address='2NFpzHCzpzhqhfFJxqpdNSj7zyAZARGa8BE', + amount=10000000, + script_type=proto_types.PAYTOADDRESS, + ) + out2 = proto_types.TxOutputType( + address_n=self.client.expand_path("44'/1'/1'/1/0"), + multisig=proto_types.MultisigRedeemScriptType( + pubkeys=[proto_types.HDNodePathType(node=deserialize(xpubs[0]), address_n=[1,0]), + proto_types.HDNodePathType(node=correcthorse, address_n=[]), + proto_types.HDNodePathType(node=correcthorse, address_n=[])], + signatures=[b'',b'',b''], + m=2, + ), + script_type=proto_types.PAYTOMULTISIG, + amount=50000000 - 10000000 - 5000, + ) + with self.client: + self.client.set_expected_responses([ + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.ButtonRequest(code=proto_types.ButtonRequest_ConfirmOutput), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.ButtonRequest(code=proto_types.ButtonRequest_ConfirmOutput), + proto.ButtonRequest(code=proto_types.ButtonRequest_SignTx), + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.TxRequest(request_type=proto_types.TXFINISHED), + ]) + (signatures1, serialized_tx) = self.client.sign_tx('Bcash Testnet', [inp1], [out1, out2]) + self.assertEqual(binascii.hexlify(signatures1[0]), b'3044022053acc56aea1d26d84c9a24dae82f2d47f0383ceec57425264f8246d4d6fa49e80220100e65ea7a62df317adf4fb61a24da6d6236a5c688fb309afd11c4262f9b4e43') + + def test_send_bcc_multisig_change(self): + self.setup_mnemonic_allallall() + self.client.set_tx_api(TxApiTestnet) + xpubs = []; + for n in map(lambda index : self.client.get_public_node(self.client.expand_path("44'/1'/"+str(index)+"'")), range(1,4)): + xpubs.append(n.xpub) + + def getmultisig(chain, nr, signatures=[b'',b'',b''], xpubs=xpubs): + return proto_types.MultisigRedeemScriptType( + pubkeys=map(lambda xpub : proto_types.HDNodePathType(node=deserialize(xpub), address_n=[chain,nr]), xpubs), + signatures=signatures, + m=2, + ) + inp1 = proto_types.TxInputType( + address_n=self.client.expand_path("44'/1'/1'/0/0"), + multisig=getmultisig(0,0), + # moUJnmge8SRXuediK7bW6t4YfrPqbE6hD7 + amount=50000000, + prev_hash=binascii.unhexlify('5473e99f750a44de54c6bdaf2de393768e386b34545d2497f9cc6f28d2440289'), + prev_index=0, + script_type=proto_types.SPENDMULTISIG, + ) + out1 = proto_types.TxOutputType( + address='2NFpzHCzpzhqhfFJxqpdNSj7zyAZARGa8BE', + amount=10000000, + script_type=proto_types.PAYTOADDRESS, + ) + out2 = proto_types.TxOutputType( + address_n=self.client.expand_path("44'/1'/1'/1/0"), + multisig=getmultisig(1,0), + script_type=proto_types.PAYTOMULTISIG, + amount=50000000 - 10000000 - 5000, + ) + with self.client: + self.client.set_expected_responses([ + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.ButtonRequest(code=proto_types.ButtonRequest_ConfirmOutput), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.ButtonRequest(code=proto_types.ButtonRequest_SignTx), + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.TxRequest(request_type=proto_types.TXFINISHED), + ]) + (signatures1, serialized_tx) = self.client.sign_tx('Bcash Testnet', [inp1], [out1, out2]) + + self.assertEqual(binascii.hexlify(signatures1[0]), b'3045022100c5d9dac04f7843ab4832f4e743ffd2f9c1a839f9c350c89613536ca04994a511022025c33450e54230e0e9dc26cdeb9de928049f03e9e74f7e3385b76af1af819b34') + + signatures1.append(b'') + signatures1.append(b'') + inp1b = proto_types.TxInputType( + address_n=self.client.expand_path("44'/1'/3'/0/0"), + multisig=getmultisig(0,0,signatures1), + # moUJnmge8SRXuediK7bW6t4YfrPqbE6hD7 + amount=50000000, + prev_hash=binascii.unhexlify('5473e99f750a44de54c6bdaf2de393768e386b34545d2497f9cc6f28d2440289'), + prev_index=0, + script_type=proto_types.SPENDMULTISIG, + ) + out2.address_n[2] = 3 + 0x80000000 + + with self.client: + self.client.set_expected_responses([ + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.ButtonRequest(code=proto_types.ButtonRequest_ConfirmOutput), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.ButtonRequest(code=proto_types.ButtonRequest_SignTx), + proto.TxRequest(request_type=proto_types.TXINPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=0)), + proto.TxRequest(request_type=proto_types.TXOUTPUT, details=proto_types.TxRequestDetailsType(request_index=1)), + proto.TxRequest(request_type=proto_types.TXFINISHED), + ]) + (signatures1, serialized_tx) = self.client.sign_tx('Bcash Testnet', [inp1b], [out1, out2]) + + self.assertEqual(binascii.hexlify(signatures1[0]), b'30450221009907e496658ae839a683b12d740669daba56d68039ba618d51ae8af359d2e67f022071d8b903dbdf8d1a9618cd3e4b056601ec5e67b070b22e19d5e6aad9092f6fae') + self.assertEqual(binascii.hexlify(serialized_tx), b'0100000001890244d2286fccf997245d54346b388e7693e32dafbdc654de440a759fe9735400000000fdfe0000483045022100c5d9dac04f7843ab4832f4e743ffd2f9c1a839f9c350c89613536ca04994a511022025c33450e54230e0e9dc26cdeb9de928049f03e9e74f7e3385b76af1af819b34414830450221009907e496658ae839a683b12d740669daba56d68039ba618d51ae8af359d2e67f022071d8b903dbdf8d1a9618cd3e4b056601ec5e67b070b22e19d5e6aad9092f6fae414c69522103bae960983f83e28fcb8f0e5f3dc1f1297b9f9636612fd0835b768e1b7275fb9d2102fda9c8db4c874fd898256a4524310fa4f66c7dccbb3dca2ded179dcae233c6c121026b27cc8adf7982277d923a5039d90236af0f8e3b27ddfcbc6da4fcc4a88b6b4753aeffffffff02809698000000000017a914f7b5778714965b2b66cbc373b739b4ab5749057d87784662020000000017a9147708c4321dd802870439fa7563716bbccabb5ba68700000000')