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.

Sign up for GitHub

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

Jump to bottom

fix: some invalid values for the v field in Ethereum signatures are accepted #163

Closed
Tracked by #131
obatirou opened this issue Nov 28, 2024 · 0 comments · Fixed by #219
Closed
Tracked by #131

fix: some invalid values for the v field in Ethereum signatures are accepted #163

obatirou opened this issue Nov 28, 2024 · 0 comments · Fixed by #219
Assignees
@obatirou
Milestone
EF tests no state...

Comments

@obatirou
Copy link
Contributor

obatirou commented Nov 28, 2024 •
edited
Loading

Issue https://github.com/code-423n4/2024-09-kakarot-findings/blob/main/data/RadiantLabs-Q.md
PR kakarot kkrt-labs/kakarot#1633
@obatirou obatirou mentioned this issue Nov 28, 2024
Closed
@github-project-automation github-project-automation bot moved this to Backlog in Keth Nov 28, 2024
@obatirou obatirou added this to the EF tests no state root milestone Nov 28, 2024
@ClementWalter ClementWalter moved this from Backlog to Todo in Keth Dec 6, 2024
@obatirou obatirou moved this from Todo to In progress in Keth Dec 6, 2024
@obatirou obatirou self-assigned this Dec 6, 2024
@obatirou obatirou mentioned this issue Dec 6, 2024
Merged
@obatirou obatirou moved this from In progress to In review in Keth Dec 6, 2024
ClementWalter pushed a commit that referenced this issue Dec 9, 2024
@obatirou
fix: ensure y_parity is 0 or 1 (#219)
16a666f 
Closes #163

Ensure y_parity is 0 or 1. Adds also missing tests.

Note to reviewer: this is a fix imported from C4 mitigation, ensure the
fix was correctly ported by looking at the corresponding issue and PR.
@github-project-automation github-project-automation bot moved this from In review to Done in Keth Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@obatirou obatirou

Labels
None yet
Projects
Keth
Archived in project
Milestone
EF tests no state root
Development

Successfully merging a pull request may close this issue.

fix: ensure y_parity is 0 or 1 obatirou/keth
1 participant
@obatirou