-
Notifications
You must be signed in to change notification settings - Fork 969
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 docs and makefile update #2246
Fuzzing docs and makefile update #2246
Conversation
docs/fuzzing.md
Outdated
the former is incompatible, and the latter doesn't interoperate with the | ||
compiler wrappers. | ||
Start with a clean workspace, `make clean` or cleaner; enable a few compilation flags **WHICH SHOULD | ||
NEVER BE ENABLED IN PRODUCTION**: `FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION` and `AFL_LLVM_MODE`; and |
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.
are you sure about this? FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
seems to be set automatically by afl.
As for AFL_LLVM_MODE
, if it's not automatically set, we should just set it when configuring with --enable-afl
in the configure script
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.
Cool, thanks for pointing this out. Did not realize FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
was set, but see it here.
Will add set AFL_LLVM_MODE
as part of the configure script.
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.
Done.
Removed part about setting AFL related env vars as, mentioned above, those are set by either afl or as part of |
r+ 385bd0e |
Fuzzing docs and makefile update Reviewed-by: MonsieurNicolas
Description
Following #2182 , this updates the fuzzing.md doc with what I have learned and the updates I have made. This also updates the shell script so that we generate a lot of inputs and then minimize them with
afl-cmin
.