[libav-devel] [PATCH 07/11] vaapi_h264: Fix VUI max_dec_frame_buffering

Luca Barbato lu_zero at gentoo.org
Wed Nov 29 23:37:46 CET 2017


On 29/11/2017 23:11, Mark Thompson wrote:
> From: Jun Zhao <jun.zhao at intel.com>
> 
> This should refer to the existing SPS structure, not the VAAPI sequence
> parameter buffer (which is not yet initialised).
> 
>  From ffmpeg commit f31478ba1472afe5c1eed60f219ae331816425a2.
> 
> Signed-off-by: Mark Thompson <sw at jkqxz.net>
> ---
>   libavcodec/vaapi_encode_h264.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
> index dedcf5b03..0c9e0c46b 100644
> --- a/libavcodec/vaapi_encode_h264.c
> +++ b/libavcodec/vaapi_encode_h264.c
> @@ -453,7 +453,7 @@ static int vaapi_encode_h264_init_sequence_params(AVCodecContext *avctx)
>       sps->vui.log2_max_mv_length_horizontal = 16;
>       sps->vui.log2_max_mv_length_vertical   = 16;
>       sps->vui.max_num_reorder_frames        = (ctx->b_per_p > 0);
> -    sps->vui.max_dec_frame_buffering       = vseq->max_num_ref_frames;
> +    sps->vui.max_dec_frame_buffering       = sps->max_num_ref_frames;
>   
>       pps->nal_unit_header.nal_ref_idc = 3;
>       pps->nal_unit_header.nal_unit_type = H264_NAL_PPS;
> 

Sounds good.


More information about the libav-devel mailing list