[SCM] libav/experimental: Prevent memory leak introduced in r22389 in Bink demuxer: pass partial packets to decoder.

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:04:12 UTC 2013


The following commit has been merged in the experimental branch:
commit 70b462cc2940bce1023adb0780a83725526117f4
Author: Peter Ross <pross at xvid.org>
Date:   Thu Mar 11 12:29:02 2010 +0000

    Prevent memory leak introduced in r22389 in Bink demuxer: pass partial packets to decoder.
    
    Originally committed as revision 22468 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavformat/bink.c b/libavformat/bink.c
index 3208d28..ac04e76 100644
--- a/libavformat/bink.c
+++ b/libavformat/bink.c
@@ -212,15 +212,16 @@ static int read_packet(AVFormatContext *s, AVPacket *pkt)
         bink->current_track++;
         if (audio_size >= 4) {
             /* get one audio packet per track */
-            if ((ret = av_get_packet(pb, pkt, audio_size)) != audio_size)
-                return ret < 0 ? ret : AVERROR(EIO);;
+            if ((ret = av_get_packet(pb, pkt, audio_size)) < 0)
+                return ret;
             pkt->stream_index = bink->current_track;
             pkt->pts = bink->audio_pts[bink->current_track - 1];
 
             /* Each audio packet reports the number of decompressed samples
                (in bytes). We use this value to calcuate the audio PTS */
-            bink->audio_pts[bink->current_track -1] +=
-                AV_RL32(pkt->data) / (2 * s->streams[bink->current_track]->codec->channels);
+            if (pkt->size >= 4)
+                bink->audio_pts[bink->current_track -1] +=
+                    AV_RL32(pkt->data) / (2 * s->streams[bink->current_track]->codec->channels);
             return 0;
         } else {
             url_fseek(pb, audio_size, SEEK_CUR);
@@ -228,9 +229,8 @@ static int read_packet(AVFormatContext *s, AVPacket *pkt)
     }
 
     /* get video packet */
-    if ((ret = av_get_packet(pb, pkt, bink->remain_packet_size))
-                                   != bink->remain_packet_size)
-        return ret < 0 ? ret : AVERROR(EIO);
+    if ((ret = av_get_packet(pb, pkt, bink->remain_packet_size)) < 0)
+        return ret;
     pkt->stream_index = 0;
     pkt->pts = bink->video_pts++;
     pkt->flags |= PKT_FLAG_KEY;

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list