trezor / trezor-core

:lock: Don't use this repo, use the new monorepo instead:
https://github.com/trezor/trezor-firmware
GNU General Public License v3.0
354 stars 204 forks source link

Enforce BIP44 prefixes for new coins #221

Closed jpochyla closed 6 years ago

jpochyla commented 6 years ago

In case of BIP44-like path, make sure it matches the coin. Do this also for non-Bitcoin-like altcoins.

prusnak commented 6 years ago

Also for BIP49 and BIP84 (for Bitcoin forks) - the same logic.

prusnak commented 6 years ago

Also check in SignMessage and others, not just SignTx.

prusnak commented 6 years ago

Implemented in https://github.com/trezor/trezor-core/commit/51df2499495cbf9337fcade365fc10c252dd1e79