Hi Leslie

On Fri, Dec 19, 2025 at 08:11:34AM +0800, LP via ffmpeg-devel wrote:
>  Hello FFmpeg developers,
> 
> This patch adds several new fuzzer targets to the tools/ directory to
> improve

can you split this in a patch per fuzzer ?

also you may want to submit it to forgejo: https://code.ffmpeg.org/


> test coverage for various components, including libavutil (AVOptions),
> libavfilter,
> muxers, network protocols, and the CLI options parsing.
> 
> It also introduces tools/run_fuzzer_loop.c, a simple harness that allows
> running
> these fuzzers in a loop without requiring a full libFuzzer environment.
> 
> I have verified that the new targets compile and run successfully.
> 
> Please find the patch attached.
> 
> Best,
> 
> Leslie

[...]

> +    // Fill frame with fuzz data if available
> +    if (size > 0) {
> +        int copy_size = size;
> +        // Naive fill, just copy into first plane
> +        if (copy_size > frame->buf[0]->size) copy_size = frame->buf[0]->size;

FFMIN()

> +        memcpy(frame->data[0], data, copy_size);
> +    }
> +

> +    ret = av_buffersrc_add_frame(src_ctx, frame);
> +    if (ret < 0) goto end;
> +
> +    // Receive output
> +    while (1) {
> +        AVFrame *out = av_frame_alloc();
> +        ret = av_buffersink_get_frame(sink_ctx, out);
> +        av_frame_free(&out);
> +        if (ret < 0) break;
> +    }

can be simplified with while(ret >= 0)




> +
> +end:

> +    if (options_str) av_free(options_str);

the if() is unneeded


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope