From dc2fc5a919608e3937137f11548f4be5ee99523d Mon Sep 17 00:00:00 2001 From: Anton Khirnov Date: Fri, 15 Dec 2023 02:25:41 +0100 Subject: [PATCH] fftools/ffmpeg_filter: only set framerate for video Otherwise an unitialized stack value would be copied to FPSConvContext. As it's then never used, it tends not to be a problem in practice, however it is UB and some compilers warn about it. --- fftools/ffmpeg_filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c index 87ab2ca976..c1c3014453 100644 --- a/fftools/ffmpeg_filter.c +++ b/fftools/ffmpeg_filter.c @@ -2017,9 +2017,9 @@ static int choose_out_timebase(OutputFilterPriv *ofp, AVFrame *frame) if (!(tb.num > 0 && tb.den > 0)) tb = frame->time_base; + fps->framerate = fr; finish: ofp->tb_out = tb; - fps->framerate = fr; ofp->tb_out_locked = 1; return 0;