[FFmpeg-devel] [PATCH] libavfilter/vf_vpp_qsv: Fix a timestamp bug when framerate is not set

Xiang, Haihao haihao.xiang at intel.com
Wed Aug 16 04:24:48 EEST 2023


On Di, 2023-08-15 at 16:34 +0800, wenbin.chen-at-intel.com at ffmpeg.org wrote:
> From: Wenbin Chen <wenbin.chen at intel.com>
> 
> If user doesn't set framerate when he creates a filter, the filter uses
> default framerate {0, 1}. This causes error when setting timebase to
> 1/framerate. Now change it to pass inlink->time_base to outlink when
> framerate is not set.
> This patch fixes ticket: #10476 #10468
> 
> Signed-off-by: Wenbin Chen <wenbin.chen at intel.com>
> ---
>  libavfilter/vf_vpp_qsv.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
> index 92ef0f1d89..c07b45fedb 100644
> --- a/libavfilter/vf_vpp_qsv.c
> +++ b/libavfilter/vf_vpp_qsv.c
> @@ -536,7 +536,10 @@ static int config_output(AVFilterLink *outlink)
>      outlink->w          = vpp->out_width;
>      outlink->h          = vpp->out_height;
>      outlink->frame_rate = vpp->framerate;
> -    outlink->time_base  = av_inv_q(vpp->framerate);
> +    if (vpp->framerate.num == 0 || vpp->framerate.den == 0)
> +        outlink->time_base = inlink->time_base;
> +    else
> +        outlink->time_base = av_inv_q(vpp->framerate);
>  
>      param.filter_frame  = NULL;
>      param.set_frame_ext_params = vpp_set_frame_ext_params;

LGTM, will apply

Thanks
Haihao



More information about the ffmpeg-devel mailing list