Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add OSS-Fuzz on Demand build steps to extract crashes from the fuzzing output corpus #13270

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

decoNR
Copy link
Collaborator

@decoNR decoNR commented Apr 30, 2025

Add build steps to download the latest LibFuzzer build and use it to extract crashes from the fuzzing output corpus.

  • Increment ood.Dockerfile to be able to copy the fuzzing output corpus directory to OOD_OUTPUT_CORPUS_DIR.
  • Use cifuzz code to get the latest LibFuzzer build filename.
  • Add a build step to download the latest LibFuzzer build from GCS.
  • Add a build step to use the latest LibFuzzer target to extract crashes from the OOD fuzzing output corpus.
  • Improve fuzzbench.py maintainability by adding GCB_WORKSPACE_DIR constant.

Related to b/401215144 .

@decoNR
Copy link
Collaborator Author

decoNR commented Apr 30, 2025

/gcbrun oss_fuzz_on_demand.py libucl --fuzzing-engine mopt --fuzz-target ucl_add_string_fuzzer

@@ -77,7 +77,7 @@ class Config:
upload_build_logs: str = None
build_type: str = None
fuzzing_engine: str = None
fuzz_target: str = ''
fuzz_target: str = None
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably be Optional[string]

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Should I change all "str = None" to "Optional[str] = None"? I checked and the only Class that uses ": str = None" is this Config, so it would be a small change.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As discussed, I made the modifications and it's ready for merge.

@decoNR
Copy link
Collaborator Author

decoNR commented May 2, 2025

/gcbrun oss_fuzz_on_demand.py libucl --fuzzing-engine mopt --fuzz-target ucl_add_string_fuzzer

@decoNR
Copy link
Collaborator Author

decoNR commented May 5, 2025

/gcbrun oss_fuzz_on_demand.py libucl --fuzzing-engine mopt --fuzz-target ucl_add_string_fuzzer

@decoNR
Copy link
Collaborator Author

decoNR commented May 6, 2025

/gcbrun oss_fuzz_on_demand.py libucl --fuzzing-engine mopt --fuzz-target ucl_add_string_fuzzer

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