search

cisco / libsrtp

Library for SRTP (Secure Realtime Transport Protocol)
Other
1.19k stars 472 forks source link

add CIFuzz Github action to workflows #616

Closed DavidKorczynski closed 1 year ago

DavidKorczynski commented 1 year ago

Add CIFuzz workflow action to have fuzzers build and run on each PR. This is a service offered by OSS-Fuzz where libsrtp already runs. In the current PR the fuzzers gets build on a pull request and will run for 300 seconds.

I noticed the CI already has a fuzz action so this is an alternative suggestion in case you're interested -- CIFuzz has a set of features that are useful e.g. only highlighting issues only if they are introduced by the given PR, using of OSS-Fuzz code corpus and more. Let me know what you think!

pabuhler commented 1 year ago

Hi @DavidKorczynski , I would be ok with this, I think the existing fuzzer task is just to verify the fuzzer builds. If no one objects and it works we can merge it soon.

pabuhler commented 1 year ago

The other work flows also run when pushing to main, should that be added here as well, as fuzzy the result of the merge is also important ?

DavidKorczynski commented 1 year ago

The other work flows also run when pushing to main, should that be added here as well, as fuzzy the result of the merge is also important ?

Yep, this sounds good to me! Done