[FFmpeg-devel] [ogg] Respect AVERROR codes returned by ogg header parsing.

[Dale Curtis]

On Wed, Nov 29, 2017 at 5:58 PM, Michael Niedermayer <michael at niedermayer.cc
> wrote:

>
> AV_EF_EXPLODE appears to be a possibility, yes
> in general demuxers should try to extract as much as possible from a
> file and not give up if some broken packet is found
>

I don't think this is a good idea; some of these errors seem fairly fatal
to the overall process. I think instead it's better to write off the sample
you have as being invalid. That said, I defer to your judgement here, so
I've produced both versions of the patch and leave it to your decision.
Chrome is now always using AV_EF_EXPLODE, so either works for our usage.

- dale
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_ogg_v1_explode.patch
Type: text/x-patch
Size: 1971 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171130/f1577ff3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_ogg_v4.patch
Type: text/x-patch
Size: 1739 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171130/f1577ff3/attachment-0001.bin>