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.

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

Floria: Add nonce and eoa check #974

Merged
merged 2 commits into from
Dec 19, 2024
Merged

Conversation

simonlechner
Copy link
Contributor

The check regarding the nonce and the non externally owned accounts have to performed in the beginning of the transaction, therefore these checks are refactored. Further, a check for the max init code size is added.

@simonlechner simonlechner added this to the Tosca EVM milestone Dec 18, 2024
@simonlechner simonlechner requested a review from LuisPH3 December 18, 2024 10:07
@simonlechner simonlechner self-assigned this Dec 18, 2024
@simonlechner simonlechner requested a review from facuMH December 18, 2024 10:30
Copy link
Contributor

@LuisPH3 LuisPH3 left a comment

Choose a reason for hiding this comment

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

This is a little more complicated than I expected, can I have some answers to the following questions?

go/processor/floria/processor.go Show resolved Hide resolved
go/processor/floria/processor.go Show resolved Hide resolved
go/processor/floria/processor.go Show resolved Hide resolved
@simonlechner simonlechner requested a review from LuisPH3 December 18, 2024 11:05
go/processor/floria/processor.go Show resolved Hide resolved
go/processor/floria/processor_test.go Show resolved Hide resolved
go/processor/floria/processor_test.go Show resolved Hide resolved
go/processor/floria/processor_test.go Show resolved Hide resolved
go/processor/floria/processor.go Show resolved Hide resolved
@simonlechner simonlechner force-pushed the simon/floria-nonce-eoa-check branch from 15dd1ab to 048acbe Compare December 18, 2024 11:40
@simonlechner simonlechner requested a review from facuMH December 18, 2024 11:42
@simonlechner simonlechner force-pushed the simon/floria-nonce-eoa-check branch from 048acbe to e3f3e51 Compare December 19, 2024 07:23
@simonlechner simonlechner merged commit 680ee56 into main Dec 19, 2024
6 checks passed
@simonlechner simonlechner deleted the simon/floria-nonce-eoa-check branch December 19, 2024 07:30
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