-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EIP-2602: Disable null hash message verification for ecrecover precompile #2602
Conversation
@@ -0,0 +1,50 @@ | |||
--- | |||
eip: 2602 | |||
title: Disable null hash message verification for ecrecover precompile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: Disable null hash message verification for ecrecover precompile | |
title: Disable null hash message verification in the ecrecover precompile |
Since you use this wording below.
|
||
## Specification | ||
|
||
In the beginning of execution of `ecrecover` precompile, fetch the message and check if it is equal to null hash (`0x0000...0000`). If so, set output length to `0` and return. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this specification a bit more extended? Should clarify if "message" here means a part of the input to the precompile or the entire input?
Also probably spelling out the entire hash is fine as that would signal how many bits it is (256?).
@sorpaas can you please respond? It would be nice to clarify the above and merge this. |
There has been no activity on this pull request for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review. |
This pull request was closed due to inactivity. If you are still pursuing it, feel free to reopen it and respond to any feedback or request a review in a comment. |
Disable null hash message
ecrecover
operation in the precompile, due to it being potentially unsafe.