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

Fuzzing: Build fuzzers from other repository #2914

Closed
wants to merge 1 commit into from

Conversation

AdamKorcz
Copy link
Contributor

This PR adds two lines to the OSS-fuzz build script that builds the fuzzers from an independent repository.

It builds the fuzzers from the following PRs:

  1. Fuzzing: Add new fuzzer #2848
  2. Libcontainer: Add new fuzzer #2858
  3. Specconv: Add fuzzer #2864
  4. Libcontainer: Add fuzzer #2872
  5. Fscommon: Add fuzzer #2878
  6. cgroups/fs2: Add fuzzer #2879

While it would be good to merge the fuzzers from the PR's above, it is not necessary to do that now. What is important is to run the fuzzers and this PR allows that without merging the PR's above now.

Once this PR is merged, the fuzzers will be built and run by OSS-fuzz continuously.

Signed-off-by: AdamKorcz <adam@adalogics.com>
@kolyshkin
Copy link
Contributor

I'd rather merge those other PRs. The problem was, I guess, in #2869, which needs to be merged first.

@kolyshkin

This comment has been minimized.

@cyphar
Copy link
Member

cyphar commented Apr 22, 2021

#2869 has been merged, which I guess means we can start reviewing and merging the others, right? (And this PR can be closed?)

@AdamKorcz
Copy link
Contributor Author

we can start reviewing and merging the others, right?

Yes. Closing.

@AdamKorcz AdamKorcz closed this Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants