[FFmpeg-devel] [PATCH 3/3] vaapi_encode_vp8: memset the the structure to 0

[Xiang, Haihao]

On Mon, 2018-05-07 at 21:48 +0100, Mark Thompson wrote:
> On 04/05/18 15:41, Haihao Xiang wrote:
> > The structure has reserved bytes, it is required to set the reserved
> > bytes to 0 for future use.
> > 
> > Signed-off-by: Haihao Xiang <haihao.xiang at intel.com>
> > ---
> >  libavcodec/vaapi_encode_vp8.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/libavcodec/vaapi_encode_vp8.c b/libavcodec/vaapi_encode_vp8.c
> > index b4c5521d1f..a2e861a8d1 100644
> > --- a/libavcodec/vaapi_encode_vp8.c
> > +++ b/libavcodec/vaapi_encode_vp8.c
> > @@ -143,6 +143,8 @@ static int
> > vaapi_encode_vp8_write_quant_table(AVCodecContext *avctx,
> >      *type     = VAQMatrixBufferType;
> >      *data_len = sizeof(quant);
> >  
> > +    memset(&quant, 0, sizeof(quant));
> > +
> >      if (pic->type == PICTURE_TYPE_P)
> >          q = priv->q_index_p;
> >      else
> > 
> 
> Yep, applied.
> 
> (... is there any plan to add anything to that structure?)
> 

Thanks for applying the patch and so far there is no plan to change that
structure. 


> Thanks,
> 
> - Mark
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel