[FFmpeg-devel] [PATCH] Abort Ogg header parsing when we encounter a data packet.

[Reimar Döffinger]

On Sat, Apr 09, 2011 at 05:54:56PM +0200, Clément Bœsch wrote:
> On Sat, Apr 09, 2011 at 05:15:27PM +0200, Reimar Döffinger wrote:
> > On Fri, Apr 08, 2011 at 10:18:56AM +0200, Clément Bœsch wrote:
> > > On Fri, Apr 08, 2011 at 01:20:59AM +0200, Reimar Döffinger wrote:
> > > > Fixes ticket #15.
> > > > ---
> > > >  libavformat/oggdec.c |    8 ++++----
> > > >  1 files changed, 4 insertions(+), 4 deletions(-)
> > > > 
> > > > diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c
> > > > index ddc7a1f..bcec2d5 100644
> > > > --- a/libavformat/oggdec.c
> > > > +++ b/libavformat/oggdec.c
> > > 
> > > This patch makes at least ffplay freeze with my MPD webradio…
> > 
> > URL?
> 
> It is private, but the issue is reproducible with
> http://stream-sd.radioparadise.com:9000/rp_192.ogg for instance.
> 
> > Could you remove the line that sets AVFMTCTX_NOHEADER
> > and test?
> 
> Yes, it seems to work now for this test case.

I pushed it, since it fixes that sample and breaks none of the
samples I have.
As always with Ogg it's quite likely I missed something anyway though...