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

Pavel Koshevoy pkoshevoy at gmail.com
Wed Feb 15 19:05:56 EET 2017


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.

    Pavel.


More information about the ffmpeg-devel mailing list