Skip to content
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

Closed
wants to merge 5 commits into from

Conversation

sorpaas
Copy link
Contributor

@sorpaas sorpaas commented Apr 15, 2020

Disable null hash message ecrecover operation in the precompile, due to it being potentially unsafe.

@sorpaas sorpaas changed the title Disable null hash message for ecrecover precompile EIP-2602: Disable null hash message for ecrecover precompile Apr 15, 2020
@sorpaas sorpaas changed the title EIP-2602: Disable null hash message for ecrecover precompile EIP-2602: Disable null hash message verification for ecrecover precompile Apr 15, 2020
@@ -0,0 +1,50 @@
---
eip: 2602
title: Disable null hash message verification for ecrecover precompile
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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.
Copy link
Member

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?).

@axic
Copy link
Member

axic commented May 16, 2020

@sorpaas can you please respond? It would be nice to clarify the above and merge this.

@github-actions
Copy link

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.

@github-actions github-actions bot added the stale label Oct 27, 2020
@github-actions
Copy link

github-actions bot commented Nov 3, 2020

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.

@github-actions github-actions bot closed this Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants