[FFmpeg-cvslog] libavformat/oggdec: Free stream private when header parsing fails.

Chris Cunningham git at videolan.org
Sun Jun 5 03:16:41 CEST 2016


ffmpeg | branch: release/3.0 | Chris Cunningham <chcunningham at chromium.org> | Mon May  9 15:27:29 2016 -0700| [069eea16d975643bee6a42643c9bac54f170ee9c] | committer: Michael Niedermayer

libavformat/oggdec: Free stream private when header parsing fails.

Leaking this private structure opens up the possibility that it may
be re-used when parsing later packets in the stream. This is
problematic if the later packets are not the same codec type (e.g.
private allocated during Vorbis parsing, but later packets are Opus
and the private is assumed to be the oggopus_private type in
opus_header()).

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
(cherry picked from commit 542f725964e52201000ec34e2f23229cf534ad3a)

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=069eea16d975643bee6a42643c9bac54f170ee9c
---

 libavformat/oggdec.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c
index 2d99b4a..8f3b44c 100644
--- a/libavformat/oggdec.c
+++ b/libavformat/oggdec.c
@@ -701,6 +701,7 @@ static int ogg_read_header(AVFormatContext *s)
         if (ogg->streams[i].header < 0) {
             av_log(s, AV_LOG_ERROR, "Header parsing failed for stream %d\n", i);
             ogg->streams[i].codec = NULL;
+            av_freep(&ogg->streams[i].private);
         } else if (os->codec && os->nb_header < os->codec->nb_header) {
             av_log(s, AV_LOG_WARNING,
                    "Headers mismatch for stream %d: "



More information about the ffmpeg-cvslog mailing list