new(tests): EOF - EIP-7620: Dangling data in subcontainer test

[shemnon]

Signed-off-by: Danno Ferrin danno@numisight.com

🗒️ Description

Test dangling data inside a subcontainer

🔗 Related Issues

✅ Checklist

• All: Set appropriate labels for the changes.
• All: Considered squashing commits to improve commit history.
• All: Added an entry to CHANGELOG.md.
• All: Considered updating the online docs in the ./docs/ directory.
• Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
• Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
• Tests: Included the type and version of evm t8n tool used to locally execute test cases: e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
• Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

[shemnon]

Covers this fuzzing find - ipsilon/eof#146 (comment)

[chfast]

Have you checked if geth is failing these tests? I believe it only should fail the initcode variant.

[shemnon]

Interestingly, geth only fails the (corrected) initcode form of the test, and correctly marks the runtime mode of the test as invalid.

[chfast]

Interestingly, geth only fails the (corrected) initcode form of the test, and correctly marks the runtime mode of the test as invalid.

This agrees with the fuzzer. I believe this is because it passes "initcode" mode to subcontainer validation and maybe this incorrectly allows additional bytes (as in create transaction).

[shemnon]

Here's the fix, explains the initcode only failure - MariusVanDerWijden/go-ethereum@4392a1c

I haven't dug in deeply but I concur with the handling hypothesis.

[marioevz]

LGTM, thanks!