From 3393cd85459cb9c23dec90373da91559987000ee Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sun, 7 Dec 2014 02:10:19 +0100 Subject: [PATCH] avcodec/utils: check AVframe.format being set in avcodec_encode_video2() We dont fail hard if its not set as the old API allowed this and our examples did in the distant past not set it, these examples still work with the current code and some encoders. Based on suggestion by: funman Signed-off-by: Michael Niedermayer --- libavcodec/utils.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 66fe62cc85..a75fe1a80e 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2088,6 +2088,9 @@ int attribute_align_arg avcodec_encode_video2(AVCodecContext *avctx, if (av_image_check_size(avctx->width, avctx->height, 0, avctx)) return AVERROR(EINVAL); + if (frame && frame->format == AV_PIX_FMT_NONE) + av_log(avctx, AV_LOG_WARNING, "AVFrame.format is not set\n"); + av_assert0(avctx->codec->encode2); ret = avctx->codec->encode2(avctx, avpkt, frame, got_packet_ptr);