Hello,

On Mon, 24. May 11:52, Tom Boshoven wrote:
> This fixes an issue where the yadif filter could cause the timebase 
> denominator to overflow.
> 
> Signed-off-by: Tom Boshoven <t...@jwplayer.com>
> ---
>  libavfilter/vf_yadif.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavfilter/vf_yadif.c b/libavfilter/vf_yadif.c
> index a6942a2aa2..b067c5eead 100644
> --- a/libavfilter/vf_yadif.c
> +++ b/libavfilter/vf_yadif.c
> @@ -293,8 +293,7 @@ static int config_output(AVFilterLink *outlink)
>      AVFilterContext *ctx = outlink->src;
>      YADIFContext *s = ctx->priv;
>  
> -    outlink->time_base.num = ctx->inputs[0]->time_base.num;
> -    outlink->time_base.den = ctx->inputs[0]->time_base.den * 2;
> +    outlink->time_base = av_mul_q(outlink->time_base, (AVRational){1, 2});
>      outlink->w             = ctx->inputs[0]->w;
>      outlink->h             = ctx->inputs[0]->h;

Some fate tests fail on patchwork:
https://patchwork.ffmpeg.org/project/ffmpeg/patch/20210524155206.47644-1-...@jwplayer.com/

-- 
Andriy
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to