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

[WIP] CMakify the fuzzing directory #583

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

Conversation

derwolfe
Copy link
Contributor

This is intended to make adding, updating, and maintaining fuzzers a bit easier.

  • It makes it so fuzzers are built and run via cmake and make and exercised by CI
  • It adds a unit test for the current fuzzer, while also making it possible to automatically add ctest tests for any new fuzzers that are added.
  • It adds documentation detailing how fuzzers can be tested via oss-fuzz tooling and how to add new testcases.

I might move build.sh back out to oss-fuzz as it seems like it is kind of here without any real context or need.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.924% when pulling 968bdd0 on derwolfe:cmakify-fuzzers into 4badbe9 on json-c:master.

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.

None yet

2 participants