[FFmpeg-devel] [PATCH] lavc/vda_h264_dec.c Fix NULL pointer dereference

Michael Niedermayer michael at niedermayer.cc
Thu Feb 16 14:22:32 EET 2017


On Wed, Feb 15, 2017 at 10:05:56AM -0700, Pavel Koshevoy wrote:
> On Thu, Feb 9, 2017 at 8:20 PM,  <pkoshevoy at gmail.com> wrote:
> > From: Pavel Koshevoy <pkoshevoy at 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.

applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170216/e1d8cbd2/attachment.sig>


More information about the ffmpeg-devel mailing list