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.xi...@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@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel