Can not sign Transaction with more then 20 BTC for PAYTOMULTISIG about connect

[lionello]

I'm running into the same issue when trying to transfer 5 BTC + 17 BTC change. I've debugged this with Chrome debugger and the inputs to signtx are correct, but the Trezor just comes back with RangeError.

[karelbilek]

Hm, this looks like it comes from some Javascript. I don't know if the bug is in Connect or Extension though To eliminate possibility that it's extension: Do you use extension or bridge for transport? Does switching to the other helps? Firefox / Chrome switch helps?

…

On Oct 9, 2017 2:20 PM, "nbphuoc" ***@***.***> wrote: I got this error when attempting to sign a transaction which output has more than 21.48 BTC: Range Error: value out of range I believe this is because 20 BTC = 2 billions satoshi and the maximum integer number is: 2,147,483,647 How do I fix this? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#82>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAGZ8eTDIeOYcrqB0TPEYNxQqCktsg9Aks5sqg95gaJpZM4PyZj2> .

[prusnak]

Also what service are you using Connect with?

[prusnak]

I was able to reproduce the issue, @Runn1ng and @szymonlesisz will look into that shortly.

[karelbilek]

OK, we found the bug, it's actually not because of the BTC, but because of segwit transactions not being properly parsed (old bitcoinjs-lib). We are looking into it

[karelbilek]

Can you try it again, with either v3 branch locally or on https://connect.trezor.io/3/ (or older versions)? It should be working now.

[lionello]

Is there a fix for trezord? It seems to fail with similar error when converting JSON to protobuf.

vendor/jsoncpp/src/lib_json/json_value.cpp:734:    
JSON_ASSERT_MESSAGE(isUInt(), "LargestInt out of UInt range");

[karelbilek]

Is the error still hapenning with Trezor Connect?

…

On Oct 17, 2017 09:17, "Lio李欧" ***@***.***> wrote: Is there a fix for trezord? It seems to fail with similar error when converting JSON to protobuf. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#82 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAGZ8VCboDFrRSvtd8V01yuwvotj5d22ks5stFSVgaJpZM4PyZj2> .

[lionello]

Changing to v3 fixed the issue! Could you share more info about the fix? Wondering whether trezord has the same or another issue.

[karelbilek]

It was an issue with segwit. When sending from an account that had even 1 transaction with segwit inputs, it threw an error. We used old bitcoinjs-lib version, that didn't know how to parse segwit transactions. It is fixed now.