Skip to content

Conversation

codchen
Copy link
Collaborator

@codchen codchen commented Sep 9, 2025

The order of the checks was wrong which would have allowed a call to modexp with baseLen == 0 && modLen == 0 post fusaka.

Also handles an edge case where base/mod/exp length >= 2**64


The order of the checks was wrong which would have allowed a call to
modexp with `baseLen == 0 && modLen == 0` post fusaka.

Also handles an edge case where base/mod/exp length >= 2**64

---------

Co-authored-by: Felix Lange <[email protected]>
Copy link

@stevenlanders stevenlanders left a comment

Choose a reason for hiding this comment

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

do we need to do anything special to ensure c.eip7823 is set so that these are enforced?

@codchen
Copy link
Collaborator Author

codchen commented Sep 10, 2025

do we need to do anything special to ensure c.eip7823 is set so that these are enforced?

yeah that'd have to be tested on the sei-chain PR that integrates with this since the config lives there

@codchen codchen merged commit 1244b84 into main Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants