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

[fuzzing-puzzles] Enable AFL fuzzing engine instead of libFuzzer + re… #1495

Merged
merged 2 commits into from Jun 8, 2018

Conversation

Dor1s
Copy link
Contributor

@Dor1s Dor1s commented Jun 7, 2018

…name the target to prevent corpus re-use.

from #1485 (comment)

Copy link
Contributor Author

@Dor1s Dor1s left a comment

Choose a reason for hiding this comment

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

In case anyone decides to merge this, please don't forget to change the weight of this project to 1.0. It's 0.0 as of now.

@@ -10,4 +10,4 @@ sanitizers:
- undefined
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jonathanmetzman do you think it's going to work fine, or should I change to ASan?

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think UBSan will work at all with AFL

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for checking, using ASan then.

@Dor1s Dor1s merged commit b9b5583 into master Jun 8, 2018
@Dor1s
Copy link
Contributor Author

Dor1s commented Jun 11, 2018

AFL made it through 112 constraints so far.
afl.html.txt

tmatth pushed a commit to tmatth/oss-fuzz that referenced this pull request Oct 22, 2018
google#1495)

* [fuzzing-puzzles] Enable AFL fuzzing engine instead of libFuzzer + rename the target to prevent corpus re-use.

* Use ASan instead of UBSan.
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.

None yet

2 participants