On Thu, Feb 9, 2017 at 8:20 PM, <pkoshe...@gmail.com> wrote: > From: Pavel Koshevoy <pkoshe...@gmail.com> > > ps.sps_list entries may be NULL, so check before dereferencing > --- > libavcodec/vda_h264_dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vda_h264_dec.c b/libavcodec/vda_h264_dec.c > index 92839e2..972bd6b 100644 > --- a/libavcodec/vda_h264_dec.c > +++ b/libavcodec/vda_h264_dec.c > @@ -226,7 +226,7 @@ static av_cold int vdadec_init(AVCodecContext *avctx) > ctx->h264_initialized = 1; > > for (i = 0; i < MAX_SPS_COUNT; i++) { > - const SPS *sps = (const SPS*)ctx->h264ctx.ps.sps_list[i]->data; > + const SPS *sps = ctx->h264ctx.ps.sps_list[i] ? (const > SPS*)ctx->h264ctx.ps.sps_list[i]->data : NULL; > if (sps && (sps->bit_depth_luma != 8 || > sps->chroma_format_idc == 2 || > sps->chroma_format_idc == 3)) { > -- > 2.6.6 >
Ping. This is a fix for a segfault I've actually run into. Pavel. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel