On 04/05/18 15:41, Haihao Xiang wrote: > Otherwise it might use unitialized last_pic in av_assert0(last_pic) > > Signed-off-by: Haihao Xiang <haihao.xi...@intel.com> > --- > libavcodec/vaapi_encode.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c > index 36c85a3815..141e50c8ad 100644 > --- a/libavcodec/vaapi_encode.c > +++ b/libavcodec/vaapi_encode.c > @@ -760,7 +760,7 @@ fail: > static int vaapi_encode_truncate_gop(AVCodecContext *avctx) > { > VAAPIEncodeContext *ctx = avctx->priv_data; > - VAAPIEncodePicture *pic, *last_pic, *next; > + VAAPIEncodePicture *pic, *last_pic = NULL, *next; > > // Find the last picture we actually have input for. > for (pic = ctx->pic_start; pic; pic = pic->next) { >
How do you hit this? last_pic should always get set. - Mark _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel