On Fri, 17 Oct 2014 01:55:35 -0500
Rodger Combs <rodger.co...@gmail.com> wrote:

> 
> > On Oct 17, 2014, at 01:52, Rodger Combs <rodger.co...@gmail.com> wrote:
> > 
> > 
> >> On Oct 17, 2014, at 01:16, Rodger Combs <rodger.co...@gmail.com 
> >> <mailto:rodger.co...@gmail.com>> wrote:
> >> 
> >> This fixes https://trac.ffmpeg.org/ticket/3934 
> >> <https://trac.ffmpeg.org/ticket/3934>, but I'm not sure if there was a 
> >> good reason for this to be here to begin with. Perhaps a protection 
> >> against infinite recursion (though I believe EBML_MAX_DEPTH serves that 
> >> purpose to some degree)?
> >> <0001-matroskadec-execute-seekheads-recursively.patch>
> > 
> > Evidently either I or my mail client screwed up and the patch didn't get 
> > attached. Whoops.
> > <0001-matroskadec-execute-seekheads-recursively.patch>
> > Let's try that again...
> 
> Welp, apparently my email client's borked badly in some way. Here's a gist 
> link instead: https://gist.github.com/08f111e72b8b5ddba078
> 

Yes, seekheads can reference other seekheads. IMO it should perhaps
keep a list of elements to read (and which have already been read),
which includes the seakhead. Since it'd be a list, it'd be safe against
recursion.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to