Re: [libav-devel] [PATCH 1/4] ogg: check that the expected number of headers had been parsed

2012-09-22 Thread Luca Barbato
On 9/22/12 3:18 AM, Justin Ruggles wrote: On 09/20/2012 03:51 AM, Luca Barbato wrote: diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c index 9860a0e..c4928d0 100644 --- a/libavformat/oggparseflac.c +++ b/libavformat/oggparseflac.c @@ -88,11 +88,13 @@ old_flac_header (AVFormat

Re: [libav-devel] [PATCH 1/4] ogg: check that the expected number of headers had been parsed

2012-09-21 Thread Justin Ruggles
On 09/20/2012 03:51 AM, Luca Barbato wrote: > diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c > index 9860a0e..c4928d0 100644 > --- a/libavformat/oggparseflac.c > +++ b/libavformat/oggparseflac.c > @@ -88,11 +88,13 @@ old_flac_header (AVFormatContext * s, int idx) > const stru

[libav-devel] [PATCH 1/4] ogg: check that the expected number of headers had been parsed

2012-09-20 Thread Luca Barbato
Not having the header for a codec is a tell-tale of a broken file. --- libavformat/oggdec.c | 13 - libavformat/oggdec.h |5 + libavformat/oggparsecelt.c |1 + libavformat/oggparsedirac.c|2 ++ libavformat/oggparseflac.c |6 --