[FFmpeg-devel] [PATCH] lavf/oggdec: check for begin-of-stream flag in case of chained streams.
ubitux at gmail.com
Sat Sep 22 00:39:50 CEST 2012
On Sat, Sep 22, 2012 at 12:26:35AM +0200, Michael Niedermayer wrote:
> On Fri, Sep 21, 2012 at 10:39:02PM +0200, Clément Bœsch wrote:
> > Fix Ticket #1617, revealing a regression I introduced in 8f3eebd.
> > ---
> > Difference from the latest attempt: as requested by Michael, I needed to make
> > sure no stream is added in between Ogg context save/restore operations (because
> > it would likely lead to a mismatch between ogg->nstreams and
> > AVFormatContext->nb_streams after the restore op). Before this patch, checking
> > for ogg->headers was preventing this: ogg->headers is always set before any ogg
> > save/restore (though, it was also preventing from creating the stream when
> > necessary).
> > But I still don't get how the ogg_new_stream() is never called between
> > ogg_save/restore calls (the ogg->state I added in ogg_new_stream is never
> > triggered with my tests), while it obviously calls the same read pages and
> > packets function as usual...
> > ---
> > libavformat/oggdec.c | 22 +++++++++++++++++++++-
> > libavformat/oggdec.h | 1 +
> > 2 files changed, 22 insertions(+), 1 deletion(-)
Thank you for your patience & review on this patch. Commit commented and
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 490 bytes
Desc: not available
More information about the ffmpeg-devel