From 1bd280f73076b7a58371585b95b08379f2445332 Mon Sep 17 00:00:00 2001 From: Baptiste Coudurier Date: Mon, 13 Apr 2009 03:38:55 +0000 Subject: [PATCH] Set frame defaults in try_decode_frame. Fix: ==22211== Conditional jump or move depends on uninitialised value(s) ==22211== at 0x811E692: ff_print_debug_info (mpegvideo.c:1137) ==22211== by 0x82E6A7B: decode_frame (h264.c:7897) ==22211== by 0x80F2823: avcodec_decode_video2 (utils.c:564) [...] ==22211== Uninitialised value was created by a stack allocation ==22211== at 0x8064AA3: try_decode_frame (utils.c:1845) Originally committed as revision 18483 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavformat/utils.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index df45f9efcc..bdf168ae93 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -1860,6 +1860,7 @@ static int try_decode_frame(AVStream *st, AVPacket *avpkt) if(!has_codec_parameters(st->codec)){ switch(st->codec->codec_type) { case CODEC_TYPE_VIDEO: + avcodec_get_frame_defaults(&picture); ret = avcodec_decode_video2(st->codec, &picture, &got_picture, avpkt); break;