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

F26: CALL & STATICCALL panic when passed invalid precompile address #1456

Closed
mriise opened this issue Jan 13, 2023 · 0 comments · Fixed by filecoin-project/builtin-actors#1042
Closed

F26: CALL & STATICCALL panic when passed invalid precompile address #1456

mriise opened this issue Jan 13, 2023 · 0 comments · Fixed by filecoin-project/builtin-actors#1042
Assignees
@mriise
Labels
Audit Issues created as a result of audit
Milestone
M2.1 (rr11) Carbo...

Comments

@mriise
Copy link
Contributor

mriise commented Jan 13, 2023

DelegateCall handles this correctly since it resolves as NotFound but we should check if it is in the reserved range first then return with nothing if precompile is invalid.
@mriise mriise added the Audit Issues created as a result of audit label Jan 13, 2023
@mriise mriise self-assigned this Jan 13, 2023
@mriise mriise linked a pull request Jan 13, 2023 that will close this issue
EVM: dont panic on invalid precompile address filecoin-project/builtin-actors#1042
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mriise mriise

Labels
Audit Issues created as a result of audit
Projects
None yet
Milestone
M2.1 (rr11) Carbonado.2
Development

Successfully merging a pull request may close this issue.

EVM: dont panic on invalid precompile address filecoin-project/builtin-actors
2 participants
@mriise @maciejwitowski