diff mbox

[FFmpeg-devel] lavc/vaapi_encode_h264: correct VUI max_dec_frame_buffering setting.

Message ID 3aeea633-1ca1-f6f5-081c-cc1f4d3ee411@gmail.com
State Accepted
Commit 4b57f064477cd39e7236897bbbb90b909e6deed2
Headers show

Commit Message

Jun Zhao Oct. 24, 2017, 5:42 a.m. UTC 
From db142742fa9adb74dea2b376db618075c86a5731 Mon Sep 17 00:00:00 2001
From: Jun Zhao <jun.zhao@intel.com>
Date: Tue, 24 Oct 2017 13:25:21 +0800
Subject: [PATCH] lavc/vaapi_encode_h264: correct VUI max_dec_frame_buffering
 setting.

vseq.max_num_ref_frames not init before the VUI
max_dec_frame_buffering setting, so use sps.max_dec_frame_buffering
in this case.

Signed-off-by: Jun Zhao <jun.zhao@intel.com>
Signed-off-by: Wang, Yi A <yi.a.wang@intel.com>
---
 libavcodec/vaapi_encode_h264.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jun Zhao Oct. 24, 2017, 6 a.m. UTC | #1 
On 2017/10/24 13:42, Jun Zhao wrote:
The commit comment is wrong, will re-submit V2 to fix the typo.
diff mbox

Patch

diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
index 9a4bd53da1..1d43e934ef 100644
--- a/libavcodec/vaapi_encode_h264.c
+++ b/libavcodec/vaapi_encode_h264.c
@@ -441,7 +441,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;