MattSkala / trezor-wallet

Bitcoin Wallet for Android with TREZOR Hardware Wallet Support
MIT License
9 stars 4 forks source link

Passphrase not working properly #2

Open dsladecka opened 6 years ago

dsladecka commented 6 years ago

We have encountered an issue while trying to use Trezor with passphrase protection. The wallet was generated correctly with a short passphrase. However, when we used a long passphrase that included spaces, the generated wallet was a completely new and empty wallet.

The generated wallet was different from the one generated using an empty passphrase.

MattSkala commented 6 years ago

Thanks for the report. Unfortunately, I haven't been able to reproduce the issue yet. By using a passphrase of 27 alphanumeric characters including spaces, the same wallet as in the web wallet is generated.

Could you please provide some more information that could help reproduce the issue? How long is the passphrase used, are there any special characters included?

Also, are you sure you entered the passphrase correctly? Currently it's a bit error-prone, as the passphrase is masked as a password and the user is not forced to confirm it by retyping.

dsladecka commented 6 years ago

Thanks for your reply. The passphrase was 23 letters long and there are no special characters. We then tried to recover a wallet with a passphrase that was just one letter long and that worked perfectly.

We tried the long passphrase with three people and two different Trezor devices. We have used both the device and the host mobile phone to type the passphrase.