On 29/11/2017 23:11, Mark Thompson wrote:
From: Jun Zhao <jun.z...@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 <s...@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.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to