[ovs-dev] [PATCH] oss-fuzz: Move oss-fuzz test harnesses and fuzzer configs to ovs source repo

Ben Pfaff blp at ovn.org
Tue Jul 10 17:33:27 UTC 2018


On Tue, Jul 10, 2018 at 10:14:41AM -0700, Ben Pfaff wrote:
> On Tue, Jul 10, 2018 at 11:10:17AM +0200, Bhargava Shastry wrote:
> > Hi Ben,
> > 
> > Thanks a lot for the new patch. A couple of comments:
> > 
> > - Could you please rename each of `tests/oss-fuzz/config/*.options` from
> > *_fuzzer.options to *_target.options. This is because oss-fuzz infra
> > requires namesake fuzzer binaries and option.
> > 
> > Example: flow_extract_fuzzer.options renamed to
> > flow_extract_target.options etc.
> > 
> > - Could you please add "tests/oss-fuzz/fuzzer.h" to the distribution. At
> > the moment, I get the following build error:
> > 
> > ```
> > The following files are in git but not the distribution:
> > tests/oss-fuzz/fuzzer.h
> > Makefile:6314: recipe for target 'dist-hook-git' failed
> > make[2]: *** [dist-hook-git] Error 1
> > make[2]: Leaving directory '/src/openvswitch'
> > Makefile:5755: recipe for target 'all-recursive' failed
> > make[1]: *** [all-recursive] Error 1
> > make[1]: Leaving directory '/src/openvswitch'
> > Makefile:3373: recipe for target 'all' failed
> > make: *** [all] Error 2
> > ```
> 
> Oops.
> 
> Here's a v2:

Never mind that one, I failed to check in some of that.

I sent it formally:
        https://patchwork.ozlabs.org/patch/942118/


More information about the dev mailing list