[FFmpeg-devel] Re: [PATCH] tools: add new fuzzers for avutil, avfilter, mux, network and cli

["Leslie P. Polzer via ffmpeg-devel" <ffmpeg-devel@ffmpeg.org>]

Hi Michael,

appreciate the prompt review and comments! I've made the changes
and put up a PR:

https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21261

Best,

Leslie

On Mon, Dec 22, 2025, at 10:53 AM, Michael Niedermayer via ffmpeg-devel wrote:
> 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
> 
> _______________________________________________
> ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
> To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org
> 
> 
> *Attachments:*
>  • signature.asc
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org