Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-12 Thread Ronald S. Bultje
Hi, On Thu, May 12, 2011 at 1:16 AM, Anton Khirnov wrote: > Prefer parse_number_or_die() over atoi()/atol() parsing for the options: > -pass, -top, -vc, and -qscale. > > Improve input validation. > > Signed-off-by: Stefano Sabatini > Signed-off-by: Anton Khirnov > --- >  ffmpeg.c |   39 +++

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-11 Thread Anton Khirnov
On Tue, 10 May 2011 09:29:47 +0200, Stefano Sabatini wrote: > On date Monday 2011-05-09 20:25:31 -0400, Ronald S. Bultje encoded: > > Hi, > > > > On Mon, May 9, 2011 at 7:09 PM, Stefano Sabatini > > wrote: > > > On date Monday 2011-05-09 18:06:51 -0400, Ronald S. Bultje encoded: > > >> Hi, > >

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-11 Thread Anton Khirnov
On Mon, 9 May 2011 18:06:51 -0400, "Ronald S. Bultje" wrote: > Hi, > > On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > > -static void opt_qscale(const char *arg) > > +static int opt_qscale(const char *opt, const char *arg) > > ??{ > > - ?? ??video_qscale = atof(arg); > > - ?? ??if (video

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-10 Thread Stefano Sabatini
On date Monday 2011-05-09 20:25:31 -0400, Ronald S. Bultje encoded: > Hi, > > On Mon, May 9, 2011 at 7:09 PM, Stefano Sabatini > wrote: > > On date Monday 2011-05-09 18:06:51 -0400, Ronald S. Bultje encoded: > >> Hi, > >> > >> On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > >> > -static v

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-09 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 7:09 PM, Stefano Sabatini wrote: > On date Monday 2011-05-09 18:06:51 -0400, Ronald S. Bultje encoded: >> Hi, >> >> On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: >> > -static void opt_qscale(const char *arg) >> > +static int opt_qscale(const char *opt, const cha

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-09 Thread Stefano Sabatini
On date Monday 2011-05-09 18:06:51 -0400, Ronald S. Bultje encoded: > Hi, > > On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > > -static void opt_qscale(const char *arg) > > +static int opt_qscale(const char *opt, const char *arg) > >  { > > -    video_qscale = atof(arg); > > -    if (video

Re: [libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-09 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2011 at 1:49 PM, Anton Khirnov wrote: > -static void opt_qscale(const char *arg) > +static int opt_qscale(const char *opt, const char *arg) >  { > -    video_qscale = atof(arg); > -    if (video_qscale <= 0 || > -        video_qscale > 255) { > +    video_qscale = parse_number_

[libav-devel] [PATCH 10/18] ffmpeg: use parse_number_and_die() when it makes sense

2011-05-09 Thread Anton Khirnov
From: Stefano Sabatini Prefer parse_number_or_die() over atoi()/atol() parsing for the options: -pass, -top, -vc, and -qscale. Improve input validation. Signed-off-by: Stefano Sabatini Signed-off-by: Anton Khirnov --- ffmpeg.c | 39 ++- 1 files changed,