Skip to content
This repository was archived by the owner on May 12, 2025. It is now read-only.

Conversation

@0x416e746f6e
Copy link

this PR ports upstream ethereum#29023:

This PR fixes an overflow which can could happen if inconsistent blockchain rules were configured. Additionally, it tries to prevent such inconsistencies from occurring by making sure that merge cannot be enabled unless previous fork(s) are also enabled.

Without this PR, it is possible to set the overrides on an eth_call over RPC and trigger a method-handler crash (not a full node crash).

Hat-tip to @pleasew8t for reporting this via the bounty via a very well-written submission.

@ferranbt ferranbt merged commit 389d537 into main Mar 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants