-
-
Notifications
You must be signed in to change notification settings - Fork 544
ci: add CIFuzz workflow #556
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
Conversation
|
@catenacyber thanks for the follow-up! So this one goes along the lines of https://google.github.io/oss-fuzz/getting-started/continuous-integration/ ? How does |
|
|
No, I can do this later. Thanks for the PR! |
@catenacyber what's your opinion, does it make sense to keep it? OTOH, if the fuzzing build fails, it seems to me that step 3 in https://google.github.io/oss-fuzz/architecture/ fails, which would lead to a notification as well? |
I think that CIFuzz is indeed a replacement. The point to keep |
OK I understand. Then maybe having build instructions in test/fuzz_server.c would be sufficient. One question @catenacyber: how is the use of test/fuzz_server wired up in https://android.googlesource.com/platform/external/oss-fuzz/+/refs/heads/upstream-master/projects/libvnc? https://android.googlesource.com/platform/external/oss-fuzz/+/refs/heads/upstream-master/projects/libvnc/build.sh does none of the env setup steps that are done in https://github.com/LibVNC/libvncserver/blob/master/.github/workflows/ci.yml#L72 ... Bit opaque to me 🤔 |
|
The environment variables are set by oss-fuzz in its Dockerfile before calling build.sh Do you want more details ? |
|
A litte bit ;-)
|
This is from its build.sh https://github.com/google/oss-fuzz/blob/master/projects/libvnc/build.sh cf Most projects name their fuzz targets with
Once the fuzzer is built, you can run it as any executable. |
Ah OK and oss-fuzz runs everything in $OUT? |
Yes (everything executable with |
CC @bk138
This enables fuzzing on all Github PRs