Verify password on transition Armored (locked) -> Secure

0x0ece / yopa-native

0 stars 2 forks source link

Verify password on transition Armored (locked) -> Secure #74

Open 0x0ece opened 6 years ago

0x0ece commented 6 years ago

Repro:

Settings: set the master password to armored. Reload the app.
Cancel mp, i.e. keep locked
Go to Settings > Master password > Secure
Check that it prompts for a passphrase
Enter a wrong passphrase

Misbehavior:

It accepts the wrong passphrase

Expected behavior:

It should check the passphrase and fail if it's wrong

0x0ece commented 6 years ago

This is minor and very obscure. We can also leave it as is. It will prob get autofixed if we do some refactoring to share some logic in Settings and GroupPrompt.