On 23/10/2020 17.07, Alexander Bulekov wrote: > generic-fuzz is not a standalone fuzzer - it requires some env variables > to be set. On oss-fuzz, we set these with some predefined > generic-fuzz-{...} targets, that are thin wrappers around generic-fuzz. > Do not make a link for the generic-fuzz from the oss-fuzz build, so > oss-fuzz does not treat it as a standalone fuzzer. > > Signed-off-by: Alexander Bulekov <alx...@bu.edu> > --- > scripts/oss-fuzz/build.sh | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/scripts/oss-fuzz/build.sh b/scripts/oss-fuzz/build.sh > index 0ce2867732..40e15985cf 100755 > --- a/scripts/oss-fuzz/build.sh > +++ b/scripts/oss-fuzz/build.sh > @@ -99,8 +99,14 @@ cp "./qemu-fuzz-i386" "$DEST_DIR/bin/" > # executable name) > for target in $(./qemu-fuzz-i386 | awk '$1 ~ /\*/ {print $2}'); > do > - ln "$DEST_DIR/bin/qemu-fuzz-i386" \ > - "$DEST_DIR/qemu-fuzz-i386-target-$target" > + # Ignore the generic-fuzz target, as it requires some environment > variables to > + # be configured. We have some generic-fuzz-{pc-q35, floppy, ...} targets > that > + # are thin wrappers around this target that set the required environment > + # variables according to predefined configs.
Just FYI, I'll rewrap this comment so that it stays within the 80 columns limit to avoid a warning from checkpatch.pl. > + if [ "$target" != "generic-fuzz" ]; then > + ln "$DEST_DIR/bin/qemu-fuzz-i386" \ > + "$DEST_DIR/qemu-fuzz-i386-target-$target" > + fi > done > > echo "Done. The fuzzers are located in $DEST_DIR" > Thomas