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

Add Atheris fuzzers #987

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Atheris fuzzers #987

wants to merge 1 commit into from

Conversation

AdamKorcz
Copy link

@AdamKorcz AdamKorcz commented Jul 29, 2022

This PR adds ClusterfuzzLite and two Atheris fuzzers for line reading.

ClusterfuzzLite will run fuzzers in the CI when PRs are made. It can be extended beyond this PR with code coverage and batch fuzzing: https://google.github.io/clusterfuzzlite/running-clusterfuzzlite/github-actions/

Signed-off-by: AdamKorcz Adam@adalogics.com

Signed-off-by: AdamKorcz <Adam@adalogics.com>
@mwouts
Copy link
Owner

mwouts commented Jul 29, 2022

Hi Adam, thank you for the PR, it looks super interesting!

I am eager to see the impact on the CI. For now it has been blocked by a flake8 check (sorry about this), we should add this commit to your PR: bc47f85 (we'll be able to see the outcome of the CI on that commit)

@mwouts
Copy link
Owner

mwouts commented Jul 30, 2022

Adam, I see that the CI has run OK (except for the codecov step which is hard to get green deterministically). This is good news, but I am also looking forward to seeing a first bug signaled by this fuzzing approach. Are we sure that the tests were execute on the CI ? How does a sample failure looks like ? Do you have recommendations on how to extend the fuzzing coverage ?

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.

2 participants