Michael Niedermayer <[email protected]> added the comment:

On Sat, Jan 30, 2010 at 01:00:28PM +0000, Carl Eugen Hoyos wrote:
> Since r20093, no colour-space is detected for this sample's video stream:

commit info of that change:

Author: reimar
Date: Wed Sep 30 11:46:48 2009
New Revision: 20093

Log:
Disable parsing for ogg streams where no ogg header was found,
if no header was found the parser was not initialized and thus will
crash when trying to use it.

Modified: trunk/libavformat/oggdec.c
==============================================================================
--- trunk/libavformat/oggdec.c  Wed Sep 30 11:43:56 2009        (r20092)
+++ trunk/libavformat/oggdec.c  Wed Sep 30 11:46:48 2009        (r20093)
@@ -477,12 +477,17 @@ static int
 ogg_read_header (AVFormatContext * s, AVFormatParameters * ap)
 {
     struct ogg *ogg = s->priv_data;
+    int i;
     ogg->curidx = -1;
     //linear headers seek from start
     if (ogg_get_headers (s) < 0){
         return -1;
     }

+    for (i = 0; i < ogg->nstreams; i++)
+        if (ogg->streams[i].header < 0)
+            ogg->streams[i].codec = NULL;
+
     //linear granulepos seek from end
     ogg_get_length (s);

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato

_____________________________________________________
FFmpeg issue tracker <[email protected]>
<https://roundup.ffmpeg.org/roundup/ffmpeg/issue1723>
_____________________________________________________

Reply via email to