-
-
Notifications
You must be signed in to change notification settings - Fork 349
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 an example fuzzer with AFL-Style UI #1501
Conversation
Due to a lack of stages/stats.rs (Because #1351 is not merged into the main branch), This commit failed to build. |
you can merge #1351 to this |
can you cargo fmt? |
I found "cargo fmt" that is executed directory in root directory dosen't make ./fuzzer formatted : ( |
yes you need to cargo fmt in each fuzzer can you edit Makefile.toml for your fuzzer and pass ci? apparently parsing stdout doesn't work out for your fuzzer. |
there is also |
04cd74f
to
6d547dc
Compare
There's still an error in the example fuzzer somewhere:
|
(timeout 11s ./${FUZZER_NAME} >fuzz_stdout.log 2>/dev/null || true) & | ||
sleep 0.2 | ||
timeout 10s ./${FUZZER_NAME} >/dev/null 2>/dev/null || true | ||
if [ -z "$(grep "corpus: 30" fuzz_stdout.log)" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please change this.
you can, for example, use the corpus that saves into the disk and count the number of entries in the corpus folder
or it won't pass the test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I already have known the root cause before you make a comment. but thank for your reminders.
b442678
to
b940e18
Compare
I don't really see any output in the UI :D |
No description provided.