[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 17/17] scripts/oss-fuzz: ignore the generic-fuzz target
From: |
Thomas Huth |
Subject: |
Re: [PATCH v7 17/17] scripts/oss-fuzz: ignore the generic-fuzz target |
Date: |
Mon, 26 Oct 2020 09:53:02 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
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 <alxndr@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
- [PATCH v7 11/17] scripts/oss-fuzz: Add crash trace minimization script, (continued)
- [PATCH v7 11/17] scripts/oss-fuzz: Add crash trace minimization script, Alexander Bulekov, 2020/10/23
- [PATCH v7 05/17] fuzz: Declare DMA Read callback function, Alexander Bulekov, 2020/10/23
- [PATCH v7 13/17] fuzz: add an "opaque" to the FuzzTarget struct, Alexander Bulekov, 2020/10/23
- [PATCH v7 12/17] fuzz: Add instructions for using generic-fuzz, Alexander Bulekov, 2020/10/23
- [PATCH v7 14/17] fuzz: add generic-fuzz configs for oss-fuzz, Alexander Bulekov, 2020/10/23
- [PATCH v7 15/17] fuzz: register predefined generic-fuzz configs, Alexander Bulekov, 2020/10/23
- [PATCH v7 16/17] scripts/oss-fuzz: use hardlinks instead of copying, Alexander Bulekov, 2020/10/23
- [PATCH v7 17/17] scripts/oss-fuzz: ignore the generic-fuzz target, Alexander Bulekov, 2020/10/23