On Tue, May 05, 2015 at 09:16:44PM +0200, Andreas Cadhalpun wrote: > On 23.04.2015 17:50, Claudio Freire wrote: > > On Thu, Apr 23, 2015 at 12:43 PM, Michael Niedermayer <michae...@gmx.at> > > wrote: > >> ok, so is the patch the correct solution or are the fields already > >> invalid before this loop and should have been checked prior ? > > > > I think either the fields are already invalid before the loop (which > > could mean an invalid file), or there is a slight bug on the loop, the > > patch just avoids infinite loop in those cases. > > > > What I'm yet unable to decide is whether it's the first or the second > > case (invalid input file vs bugged loop). I'll have to carefully > > review the specs for that, and sadly I only have drafts. > > Did you get any further with this? > > If not, I think it would be good to apply the patch I sent. > The code can always be changed later if a better solution is found.
applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I do not agree with what you have to say, but I'll defend to the death your right to say it. -- Voltaire
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
