[SCM] libav/experimental: 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)

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 16:44:01 UTC 2013


The following commit has been merged in the experimental branch:
commit 1bd280f73076b7a58371585b95b08379f2445332
Author: Baptiste Coudurier <baptiste.coudurier at gmail.com>
Date:   Mon Apr 13 03:38:55 2009 +0000

    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

diff --git a/libavformat/utils.c b/libavformat/utils.c
index df45f9e..bdf168a 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;

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list