ogg: Always alloc the private context in vorbis_header
authorLuca Barbato <lu_zero@gentoo.org>
Tue, 13 Aug 2013 05:28:41 +0000 (07:28 +0200)
committerLuca Barbato <lu_zero@gentoo.org>
Tue, 13 Aug 2013 12:04:55 +0000 (14:04 +0200)
It is possible to have an initial broken header and then valid packets.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
libavformat/oggparsevorbis.c

index db462fc..adf8f03 100644 (file)
@@ -212,15 +212,15 @@ vorbis_header (AVFormatContext * s, int idx)
     struct oggvorbis_private *priv;
     int pkt_type = os->buf[os->pstart];
 
     struct oggvorbis_private *priv;
     int pkt_type = os->buf[os->pstart];
 
-    if (!(pkt_type & 1))
-        return 0;
-
     if (!os->private) {
         os->private = av_mallocz(sizeof(struct oggvorbis_private));
         if (!os->private)
             return 0;
     }
 
     if (!os->private) {
         os->private = av_mallocz(sizeof(struct oggvorbis_private));
         if (!os->private)
             return 0;
     }
 
+    if (!(pkt_type & 1))
+        return 0;
+
     if (os->psize < 1 || pkt_type > 5)
         return -1;
 
     if (os->psize < 1 || pkt_type > 5)
         return -1;