Info

Mad Panda
Author
Download ZIP (32.8 KB)

Testing and Issues

You can test this entry and submit issues during the testing period of the Blockchain Contest contest.

Entries with serious issues will not be able to win the contest, but even minor issues might be important for overall results.

Voting

25

Comments

Mad Panda Oct 16, 2019 at 11:19
The multi-sign-wallet
Also, I have created the prototype to transform the python to Fift code to simplify code implementation and move all things to a more abstract layer. If someone will look at this and left some comments it will be good and I appreciate it. Thanks!
https://github.com/Tynik/python-fift
You have not added any comments yet...
by rating

Issues

Magic Python Oct 23, 2019 at 11:28
Is it possible to create a wallet with n=16, k=10?
I see no way to pass n to the new-multi-sign-wallet script.
10
Mad Panda Oct 23, 2019 at 20:09
You can only pass the number of signatures which will be created. It's a param <number_signs>
Magic Python Oct 29, 2019 at 18:16
I don't this may work at all. The hash contains the signature, so the signature should depend on itself.

Could you please, show some address of your wallet with some transactions in it?
10
Funny Deer Oct 16, 2019 at 15:41
>"It's signed and with the list of orders also"
Sorry. I check it again and its really signed
2
Funny Deer Oct 16, 2019 at 11:57
seqno not signed - can apply reply attack
1
Mad Panda Oct 16, 2019 at 13:24
It's signed and with the list of orders also
Funny Deer Oct 16, 2019 at 11:57
accept_message() before signature check
Mad Panda Oct 16, 2019 at 13:34
Thank you, so it can be accepted when the new signature successfully checked
Chic Dolphin Oct 16, 2019 at 12:38
I could not make your wallet work. getaccount seqno is crashing. But I can see that it's possible to drain your wallet out of funds by simply submitting empty dictionaries. https://pastebin.com/63H80rEk
Mad Panda Oct 16, 2019 at 13:29
seqno was renamed on seq_no method id. Thank you for empty dict
Nobody added any issues yet...