Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

Signing tx with 3+ outputs fails #36

Open
ferib opened this issue Oct 11, 2023 · 0 comments
Open

Signing tx with 3+ outputs fails #36

ferib opened this issue Oct 11, 2023 · 0 comments

Comments

@ferib
Copy link

ferib commented Oct 11, 2023

Hi, I forked the project and modified the CanSignTransactionStandardModeConcurrently in NanoSTests to have 3 outputs.

In test CanSignMultipleTransactionOutputs at ferib@8440518

When I sign the transaction, I see my Ledger X device prompt the suspicious path a couple of times, but that's it. I do not get any prompt to verify/sign each input & output. This problem only seems to happen when the output is 3 or more. e.g.:

// NOTE: having 3+ will promt for suspicious path, but won't sign anything :( 
            spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qc8h9tmkejfzzky79euxx5acmv9xthmcnk9df0m", Network.Main)));
            spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qve2w630azhahrhtmu047prnjjzxy2rymtd06na", Network.Main)));
            spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qdj59eexd2ggf4qa7u4n3fx9anurs5ad92d2jp3", Network.Main)));

When I comment some out, it all works fine

// NOTE: having 3+ will promt for suspicious path, but won't sign anything :( 
           spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qc8h9tmkejfzzky79euxx5acmv9xthmcnk9df0m", Network.Main)));
           //spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qve2w630azhahrhtmu047prnjjzxy2rymtd06na", Network.Main)));
           //spending.Outputs.Add(new TxOut(Money.Coins(1m), BitcoinAddress.Create("bc1qdj59eexd2ggf4qa7u4n3fx9anurs5ad92d2jp3", Network.Main)));
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant