Skip to content
This repository has been archived by the owner on Apr 16, 2019. It is now read-only.

Something wrong with ethereum msg signing #312

Closed
kvhnuke opened this issue Mar 6, 2018 · 1 comment
Closed

Something wrong with ethereum msg signing #312

kvhnuke opened this issue Mar 6, 2018 · 1 comment

Comments

@kvhnuke
Copy link

kvhnuke commented Mar 6, 2018

PrivKey: 0df7c2cb84b7022491bd48a1e3217f66404850e399f72ab5d2a439e4747ec3ed
Address: 0x661a35390053d0de80caea5e65d81fd7e6a79e2e
Msg: abcdef

signed through Trezor
Sig: 0x8eb929a9ee288d84f7dc458a3974b980e499f76b33be66e9ce4016e918089cdb0e17b3c1529e90d6ad6bbed19dca26fd36f9b6f1580320c359d0b58db4f40a5d1c

signed through MEW and geth
Sig: 0xf60b8e7b6b33798e0666eff3000f0f0bd442dfe820660b4aee2e863b2a70611c1f2e78066827c233a28662cbb638493127248374d0657a9a8e761a9f9927d4e51b

Please look into this and let me know
@prusnak
Copy link
Member

prusnak commented Mar 6, 2018 •
edited
Loading

Sign/Verify message is broken on Ethereum; see ethereum/go-ethereum#14794

This won't be fixed unless there is a consensus in Ethereum community; maybe EIP-712? ethereum/EIPs#712

@prusnak prusnak closed this as completed Mar 6, 2018
kvhnuke added a commit to MyEtherWallet/etherwallet that referenced this issue Mar 6, 2018
@kvhnuke
msg signer/verifier fixed for Trezor trezor/trezor-mcu#312
00a0519
kvhnuke added a commit to MyEtherWallet/etherwallet that referenced this issue Mar 6, 2018
@kvhnuke
msg signer/verifier fixed for Trezor trezor/trezor-mcu#312
cbdffd1
@rmeissner rmeissner mentioned this issue Aug 15, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@prusnak @kvhnuke