Re: [PATCH v7 17/17] scripts/oss-fuzz: ignore the generic-fuzz target

[Darren Kenny <darren.kenny@oracle.com>]

On Friday, 2020-10-23 at 11:07:46 -04, 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 <alxndr@bu.edu>

Reviewed-by: Darren Kenny <darren.kenny@oracle.com>

> ---
>  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.
> +    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"
> -- 
> 2.28.0