Skip to content

Fix crash when calling an unimplemented precompile - #1093

Open
msooseth wants to merge 2 commits into
mainfrom
fix-unimplemented-precompile
Open

msooseth wants to merge 2 commits into
mainfrom
fix-unimplemented-precompile

Conversation

@msooseth

@msooseth msooseth commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Calls to recognized but unimplemented precompiles (0x0a-0x11, 0x100) forced costOfPrecompile, which has no case for them and raised an internalError. Dispatch them to notImplemented before computing the gas cost, so they fail with NonexistentPrecompile / PrecompileMissing.

Closes #1084

Checklist

  • tested locally
  • added automated tests
  • updated the docs
  • updated the changelog

Calls to recognized but unimplemented precompiles (0x0a-0x11, 0x100)
forced costOfPrecompile, which has no case for them and raised an
internalError. Dispatch them to notImplemented before computing the
gas cost, so they fail with NonexistentPrecompile / PrecompileMissing.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@msooseth
msooseth requested a review from arcz September 22, 2026 10:01

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant