-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Build OSS-Fuzz projects with FuzzBench fuzzers. #13126
base: oss_fuzz_on_demand_trigger
Are you sure you want to change the base?
Conversation
/gcbrun oss_fuzz_on_demand.py libaom hermes zstd --sanitizer coverage --fuzzing-engine libfuzzer |
/gcbrun oss_fuzz_on_demand.py skcms --sanitizer coverage --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms |
/gcbrun oss_fuzz_on_demand.py skcms --sanitizer coverage --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --sanitizer coverage --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --sanitizer coverage --fuzzing-engine mopt |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --sanitizer coverage --fuzzing-engine mopt |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine mopt |
1 similar comment
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine mopt |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine libafl |
/gcbrun oss_fuzz_on_demand.py skcms --fuzzing-engine eclipser |
I think this description would be improved if it explained why certain things were done, not just that they were done. |
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.
lgtm
@@ -18,9 +18,14 @@ | |||
import sys | |||
import logging | |||
|
|||
import fuzzbench, build_project |
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.
We do imports on separate lines, per the style guide.
infra/build/functions/gcb.py
Outdated
@@ -85,7 +85,8 @@ def exec_command_from_github(pull_request_number, repo, branch): | |||
logging.info('Command: %s.', command) | |||
|
|||
if command_file == OSS_FUZZ_ON_DEMAND_COMMAND_STR.split(' ')[1]: | |||
return oss_fuzz_on_demand.oss_fuzz_on_demand_main(command) | |||
return True if oss_fuzz_on_demand.oss_fuzz_on_demand_main( |
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.
Change to oss_fuzz_on_demand.oss_fuzz_on_demand_main( command) == 0
return parser.parse_args() | ||
parser.add_argument('--fuzzing-engine', | ||
required=False, | ||
default='libafl', |
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.
This is not a good default, it should probably be libfuzzer.
/gcbrun oss_fuzz_on_demand.py skcms |
Use fuzzbench.py for building OSS-Fuzz projects with FuzzBench fuzzers.
fuzzbench.py
'sget_build_steps
function in theoss_fuzz_on_demand.py
main function.FUZZ_TARGET
environment variable value to an empty string to be able to find binary targets during the build.OUT
environment variable overwrite onfuzzbench.py
to make code more understandable.Related to b/401215144 .