Skip to content

fix: BIP-39 per-word validation during cipher recovery#235

Open
BitHighlander wants to merge 3 commits into
developfrom
fix/bip39-recovery
Open

fix: BIP-39 per-word validation during cipher recovery#235
BitHighlander wants to merge 3 commits into
developfrom
fix/bip39-recovery

Conversation

@BitHighlander
Copy link
Copy Markdown
Owner

Summary

  • Validate each word against the BIP-39 wordlist as it is entered during cipher recovery
  • Widen prev_info buffer from 16 to 24 bytes to silence -Wformat-truncation
  • Prevents users from completing recovery with an invalid mnemonic word

Test plan

  • CI green
  • Enter invalid word during recovery — device rejects before moving forward

1. Per-word rejection: invalid BIP-39 words rejected immediately during
   cipher entry with "Word not found in BIP39 wordlist" + OLED warning
2. Condition flip: !enforce_wordlist -> enforce_wordlist at finalization
3. Previous word display: shows "(N.word)" during cipher entry
4. layout_cipher: added prev_word_info parameter

C31 test should now PASS with OLED screenshot.
deps/device-protocol → 8ef74da (feat(ripple): memo + THORChain routing)
deps/python-keepkey  → bf870e6 (7.14.2: XRP memo + EVM depositWithExpiry + msg-signing)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant