On Thu, Feb 9, 2017 at 8:20 PM, <[email protected]> wrote:
> From: Pavel Koshevoy <[email protected]>
>
> 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
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
