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.

Sign up for GitHub

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

Jump to bottom

EOF validation fuzzing issues #146

Closed
chfast opened this issue Jul 24, 2024 · 39 comments
Closed

EOF validation fuzzing issues #146

chfast opened this issue Jul 24, 2024 · 39 comments
Assignees
@chfast

Comments

@chfast
Copy link
Member

chfast commented Jul 24, 2024

EOF validation fuzzing issues / missing test cases found by fuzzers.
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
This was referenced Jul 25, 2024
Merged
Merged
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@shemnon shemnon mentioned this issue Jul 29, 2024
Merged
8 tasks
@chfast

This comment was marked as outdated.

Sign in to view
@chfast
Copy link
Member Author

chfast commented Jul 30, 2024

Besu stack overflow.
git: e57c811e472b7f9fc4d229ac5c9fd30983c9de52
crash.txt

@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@rakita

This comment was marked as outdated.

Sign in to view
@chfast

This comment was marked as resolved.

Sign in to view
@shemnon

This comment was marked as outdated.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@pdobacz

This comment was marked as outdated.

Sign in to view
@shemnon

This comment was marked as outdated.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@jochem-brouwer
Copy link
Contributor

How can I add EthereumJS to this fuzzing process? Should I provide an entrypoint?

@shemnon

This comment was marked as resolved.

Sign in to view
@jochem-brouwer
Copy link
Contributor

Ok, great, I will implement such interface and will report back 😄

@jochem-brouwer
Copy link
Contributor

Do you have a code example of such interface, or a full description of this interface? Such that we can add EthereumJS in one-go without having to go back and forth a lot to fix any interface bugs 😄

@shemnon

This comment was marked as resolved.

Sign in to view
@jochem-brouwer jochem-brouwer mentioned this issue Aug 3, 2024
Merged
@jochem-brouwer
Copy link
Contributor

@shemnon could you check if this works? 😄 ethereumjs/ethereumjs-monorepo#3553

@shemnon

This comment has been minimized.

Sign in to view
@shemnon

This comment was marked as resolved.

Sign in to view
@chfast

This comment was marked as outdated.

Sign in to view
@jochem-brouwer jochem-brouwer mentioned this issue Aug 7, 2024
Merged
@chfast chfast self-assigned this Aug 19, 2024
@pdobacz pdobacz mentioned this issue Aug 22, 2024
Merged
8 tasks
@shemnon

This comment has been minimized.

Sign in to view
@shemnon

This comment has been minimized.

Sign in to view
@chfast chfast mentioned this issue Sep 9, 2024
Merged
8 tasks
@ipsilon ipsilon deleted a comment from shemnon Sep 9, 2024
@chfast

This comment has been minimized.

Sign in to view
@shemnon

This comment has been minimized.

Sign in to view
@shemnon shemnon mentioned this issue Sep 16, 2024
Merged
8 tasks
@chfast
Copy link
Member Author

chfast commented Sep 30, 2024

Was EthereumJS added to your fuzzer @shemnon ?

@shemnon
Copy link
Contributor

shemnon commented Sep 30, 2024

It can run it. Any CLI launched client that supports the tx fuzz protocol for eof validation can be used.
https://github.com/hyperledger/besu/blob/main/testfuzz/build.gradle#L99-L104

@chfast
Copy link
Member Author

chfast commented Oct 1, 2024

I'm closing this because all the issues found / test cases have been sent to EEST. Open new issue if there are more things from fuzzing in the future.

@chfast chfast closed this as completed Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chfast chfast

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@shemnon @chfast @pdobacz @rakita @jochem-brouwer