On Fri, Dec 01, 2017 at 11:16:37AM -0800, Dale Curtis wrote: > On Thu, Nov 30, 2017 at 5:49 PM, Michael Niedermayer <mich...@niedermayer.cc > > wrote: > > > I dont see anything really wrong with the file > > > > For kicks, I tried running it through oggz-validate, but it doesn't know > how to handle ogm. Which TIL, is distinct from ogv. > > > > > > it seems what happens is that theres a data packet in one stream that > > preceeds the headers on the other streams, technically that data packet > > likely contains the video headers so its kind of a header too. > > > > The demuxer stops header parsing prematurly due to this. > > from there on it gets confused reading the same header twice > > while it determines the duration of the file which triggers the error > > > > There are a few differnt ways to fix this, iam not sure which is the > > simplest but i dont think the demuxer should fail in this case > > > > > Applying a workaround similar to what you did here works: > http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=c04c43b3e423d0426162828e7b180e4d0014a3f7 > > I.e. condition AVERROR_INVALIDATA based on priv->vp being incomplete here: > http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/oggparsevorbis.c;h=65b1998a02d92d0fcaeb740d8f4523641502dbea;hb=HEAD#l319 > > WDYT? Here's a patch to do this and fail on AVERROR w/o the AV_EF_EXPLODE > restriction.
will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB "You are 36 times more likely to die in a bathtub than at the hands of a terrorist. Also, you are 2.5 times more likely to become a president and 2 times more likely to become an astronaut, than to die in a terrorist attack." -- Thoughty2
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
