sysutils/ffmpeg
1
0
Fork 0
mirror of https://git.ffmpeg.org/ffmpeg.git synced 2026-10-24 02:09:03 +03:00
Code Activity

avfilter/fruc_vulkan: fall back to a plain timebase when reduction degenerates

Browse Source 
The output timebase is derived, as in vf_fps / vf_framerate, by reducing the
source timebase against the requested frame rate. That reduction can collapse to
a zero time base - av_reduce() bounds its result, so the numerator can underflow
to zero, leaving 0/1 - when the source timebase shares no useful factors with
the requested rate, leaving an unusable output timebase that breaks the
downstream PTS rescaling.

Detect the degenerate result and fall back to the plain 1/fps timebase derived
directly from the requested rate, so config_output() always yields a valid
timebase.
This commit is contained in:
Philip Langdale
2026-06-28 15:37:21 -07:00
parent 642135d007
commit 0cb9bd6da3
1 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
libavfilter/vf_fruc_vulkan.c
View File

@@ -464,6 +464,17 @@ static int config_output(AVFilterLink *outlink)
(int64_t)s->srce_time_base.den * s->dest_frame_rate.den ),
(int64_t)s->srce_time_base.den * s->dest_frame_rate.num, INT_MAX);
/* The source-timebase-derived reduction above can collapse to a zero time
* base (av_reduce() bounds its result, so the numerator can underflow to
* zero, leaving 0/1) when the source timebase shares no useful factors with
* the requested rate, which would make the output timebase unusable. Fall
* back to the plain 1/fps timebase in that case so a valid timebase is
* always produced. */
if (!s->dest_time_base.num || !s->dest_time_base.den) {
exact = av_reduce(&s->dest_time_base.num, &s->dest_time_base.den,
s->dest_frame_rate.den, s->dest_frame_rate.num, INT_MAX);
}
av_log(ctx, AV_LOG_INFO,
"time base:%u/%u -> %u/%u exact:%d\n",
s->srce_time_base.num, s->srce_time_base.den,