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

Incorrect post state for doubleSelfdestructTest_d5g0v0_Cancun #1318

Closed
hratoanina opened this issue Nov 1, 2023 · 4 comments
Closed

Incorrect post state for doubleSelfdestructTest_d5g0v0_Cancun #1318

hratoanina opened this issue Nov 1, 2023 · 4 comments

Comments

@hratoanina
Copy link


Is the post state correct here? From what I understand from the bytecode, the recipient of the selfdestruct call is supposed to be the c0de account itself (which seems to match the post state of the other fork variants), but for Cancun the balance seems to have been sent to the 1001 account instead.

@winsvega
Copy link
Collaborator

winsvega commented Mar 7, 2024

@hratoanina
is this resolved now?

@winsvega
Copy link
Collaborator

@hratoanina

@hratoanina
Copy link
Author

Sorry, I can't run the test myself at the moment, but my concerns are the same. The bytecode and the data of the transaction is the same as the other variants, but the funds seem to be transferred to account 1001 (instead of doing nothing according to EIP-6780.

@Nashtare
Copy link

Nashtare commented Apr 2, 2024

@winsvega It seems fixed since the test regeneration! (working with @hratoanina)
Thanks!

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

No branches or pull requests

3 participants