[FFmpeg-devel] [PATCH] lavf/oggdec: check for begin-of-stream flag in case of chained streams.
michaelni at gmx.at
Sat Sep 22 00:26:35 CEST 2012
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
> 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(-)
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I have often repented speaking, but never of holding my tongue.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel