On Tue, 12 Jul 2011 12:18:37 -0700, "Ronald S. Bultje" <[email protected]> wrote: > Hi, > > On Tue, Jul 12, 2011 at 9:32 AM, Anton Khirnov <[email protected]> wrote: > > --- > > ffmpeg.c | 6 +----- > > 1 files changed, 1 insertions(+), 5 deletions(-) > > > > diff --git a/ffmpeg.c b/ffmpeg.c > > index ba5859d..1f339cb 100644 > > --- a/ffmpeg.c > > +++ b/ffmpeg.c > > @@ -102,7 +102,6 @@ typedef struct ChapterMap { > > static const OptionDef options[]; > > > > #define MAX_FILES 100 > > -#define MAX_STREAMS 1024 /* arbitrary sanity check value */ > > > > static const char *last_asked_format = NULL; > > static double *ts_scale; > > @@ -3081,9 +3080,6 @@ static int opt_input_ts_scale(const char *opt, const > > char *arg) > > p++; > > scale= strtod(p, &p); > > > > - if(stream >= MAX_STREAMS) > > - ffmpeg_exit(1); > > - > > ts_scale = grow_array(ts_scale, sizeof(*ts_scale), &nb_ts_scale, stream > > + 1); > > ts_scale[stream] = scale; > > This is a poor man's attempt to prevent us from failing because the > array grows too big. An alloc check would be nice. >
grow_array() already does that. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
