[FFmpeg-devel] [PATCH] configure: Update ossfuzz stuff to clang-12+
Kacper Michajlow
kasper93 at gmail.com
Wed Jul 16 12:58:14 EEST 2025
On Tue, 15 Jul 2025 at 00:24, Michael Niedermayer
<michael at niedermayer.cc> wrote:
>
> ---
> configure | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index fc082d5467e..e568eed55d3 100755
> --- a/configure
> +++ b/configure
> @@ -4606,8 +4606,8 @@ set >> $logfile
> test -n "$valgrind" && toolchain="valgrind-memcheck"
>
> enabled ossfuzz && ! echo $CFLAGS | grep -q -- "-fsanitize=" && ! echo $CFLAGS | grep -q -- "-fcoverage-mapping" &&{
> - add_cflags -fsanitize=address,undefined -fsanitize-coverage=trace-pc-guard,trace-cmp -fno-omit-frame-pointer
> - add_ldflags -fsanitize=address,undefined -fsanitize-coverage=trace-pc-guard,trace-cmp
> + add_cflags -fsanitize=fuzzer,address,undefined -fsanitize-memory-track-origins -fno-omit-frame-pointer
> + add_ldflags -fsanitize=address,undefined
> }
How about we remove this block of code? Or move it to
--toolchain=clang-fuzz for local build? In oss-fuzz build environment
(docker) this flags shouldn't be injected (and are not because of grep
checks) and instead CFLAGS are used for compile flags and
--libfuzzer for linking.
- Kacper
More information about the ffmpeg-devel
mailing list