Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-22 Thread Rodger Combs
On Oct 21, 2014, at 19:19, Carl Eugen Hoyos ceho...@ag.or.at wrote: Rodger Combs rodger.combs at gmail.com writes: This fixes 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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Rodger Combs
On Oct 19, 2014, at 23:34, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 17:40, Michael Niedermayer michae...@gmx.at mailto:michae...@gmx.at wrote: On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Carl Eugen Hoyos
Rodger Combs rodger.combs at gmail.com writes: This fixes 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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 01:34:16AM -0500, Rodger Combs wrote: On Oct 19, 2014, at 23:34, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 17:40, Michael Niedermayer michae...@gmx.at mailto:michae...@gmx.at wrote: On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-19 Thread Rodger Combs
On Oct 17, 2014, at 17:40, Michael Niedermayer michae...@gmx.at wrote: On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: On Oct 17, 2014, at 01:52, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17,

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Rodger Combs
On Oct 17, 2014, at 01:16, Rodger Combs 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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Rodger Combs
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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread wm4
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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs 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

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs 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