]> git.sesse.net Git - ffmpeg/commitdiff
vaapi_encode: Do not zero access unit structures
authorMark Thompson <sw@jkqxz.net>
Mon, 25 Feb 2019 23:24:23 +0000 (23:24 +0000)
committerMark Thompson <sw@jkqxz.net>
Mon, 25 Feb 2019 23:27:11 +0000 (23:27 +0000)
Following b8c45bbcbc207293f955e838ea66106f4b65b1ac they contain allocated
unit arrays which will get leaked.  These operations were inconsistently
applied and never actually needed (the old uninit left them in the correct
state), so just drop them entirely.

libavcodec/vaapi_encode_h264.c
libavcodec/vaapi_encode_h265.c

index 9f984ec60d9da1ae31519eca20569d657e7382c1..0be29d5b05f9fe074f69c5dea191ec35fa8279b5 100644 (file)
@@ -299,9 +299,6 @@ static int vaapi_encode_h264_init_sequence_params(AVCodecContext *avctx)
     VAEncSequenceParameterBufferH264 *vseq = ctx->codec_sequence_params;
     VAEncPictureParameterBufferH264  *vpic = ctx->codec_picture_params;
 
-    memset(&priv->current_access_unit, 0,
-           sizeof(priv->current_access_unit));
-
     memset(sps, 0, sizeof(*sps));
     memset(pps, 0, sizeof(*pps));
 
@@ -624,9 +621,6 @@ static int vaapi_encode_h264_init_picture_params(AVCodecContext *avctx,
     VAEncPictureParameterBufferH264 *vpic = pic->codec_picture_params;
     int i;
 
-    memset(&priv->current_access_unit, 0,
-           sizeof(priv->current_access_unit));
-
     if (pic->type == PICTURE_TYPE_IDR) {
         av_assert0(pic->display_order == pic->encode_order);
 
index 9e32cd285bc38c6f40e7c1caa5a7f5b04240f932..d37100c52de3d79cccc58d20291a5aef46f6a9af 100644 (file)
@@ -270,9 +270,6 @@ static int vaapi_encode_h265_init_sequence_params(AVCodecContext *avctx)
     int chroma_format, bit_depth;
     int i;
 
-    memset(&priv->current_access_unit, 0,
-           sizeof(priv->current_access_unit));
-
     memset(vps, 0, sizeof(*vps));
     memset(sps, 0, sizeof(*sps));
     memset(pps, 0, sizeof(*pps));