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

OSSFuzz Integration PR #811

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

capuanob
Copy link

@capuanob capuanob commented Feb 4, 2024

Good evening!

This PR introduces three major changes:

  1. The addition of a CMake option ENABLE_FUZZ_TESTING and the integration of OSSFuzz harnesses into the build system
  2. Various initial fuzzing harnesses (along with a corpus), with extensibility for future targets
  3. A build script that will be called by the Dockerfile to prepare the targets for fuzzing

Once this is merged, I will be able to make a PR to OSSFuzz to host the Dockerfile that will build these harnesses nightly.

Thank you for your time and your review!

@HowardHinnant
Copy link
Owner

I'm so sorry. My understanding was that this effort would not require changes to this repo, which I don't have the time or expertise to support.

I very much appreciate the work you've done to create this PR. I would prefer you apply it to your fork of this repo.

@capuanob
Copy link
Author

capuanob commented Feb 8, 2024

@HowardHinnant

Good evening,

I have no issue supporting this PR in whatever way is needed. However, this is a very hands-off thing once it is implemented. Should you want to review any crashes that ClusterFuzz uncovers, that will always be available so long as this repo is part of the OSSFuzz program.

Since my fork doesn't have a user-base and isn't guaranteed to be up-to-date with this repo, OSSFuzz wouldn't be interested in supporting my fork as a fuzzed project.

If there's anything I can do to make this process as easy as possible, please let me know! If this requires any attention, I am listed as the POC anyways.

I would greatly appreciate if you could reconsider, as I spent a good amount of time in integrating this project based on our initial conversation.

@capuanob
Copy link
Author

@HowardHinnant Good morning, reaching back out to see if this has been given a second thought. I appreciate your time and consideration!

@HowardHinnant
Copy link
Owner

Thank you for the time that you put into this. At this time I am only accepting bug fixes to the C++ header and source files. My apologies for not making that clear before you began this work.

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