[FFmpeg-devel] [PATCH] Fix Ogg data_offset computation.
Mon Nov 22 17:52:12 CET 2010
Ping. Can I get a review for this patch please.
On Thu, Nov 18, 2010 at 3:04 PM, Aaron Colwell <acolwell at chromium.org>wrote:
> If (in your example) only the first audio packet is demuxed and then
>> a seek is done, the video packet will never be demuxer and thus
>> data_offset will stay wrong.
> You are right. I've attached a new patch that makes sure that
> data_offset is set properly before ogg_get_headers() returns. This
> should prevent the scenario you described.
> A proper fix means you have to set data_offset already when the
>> first packet is read/parsed.
>> Concerning "valid Ogg file or not" I can only think, valid it may be,
>> it just would be nice to find something one could call a "sane Ogg file".
> I agree that it would be nice to have a stronger definition of "sane", but
> unfortunately stuff like this appears to be par for the course in ogg.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 896 bytes
Desc: not available
More information about the ffmpeg-devel