[FFmpeg-devel] [PATCH] lavu/opt: fix range check logic in set_format()

Stefano Sabatini stefasab at gmail.com
Thu Dec 26 11:51:25 CET 2013


On date Monday 2013-12-16 17:03:26 +0100, Michael Niedermayer encoded:
> On Mon, Dec 16, 2013 at 04:04:11PM +0100, Stefano Sabatini wrote:
> > In particular, allow to reject undefined values. Previously the code
> > was only accepting values in the range -1 .. NB_FORMATS-1.
> > ---
> >  libavutil/opt.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/libavutil/opt.c b/libavutil/opt.c
> > index f6aa496..9f531e4 100644
> > --- a/libavutil/opt.c
> > +++ b/libavutil/opt.c
> > @@ -559,8 +559,8 @@ static int set_format(void *obj, const char *name, int fmt, int search_flags,
> >      } else
> >  #endif
> >      {
> > -        min = FFMIN(o->min, -1);
> > -        max = FFMAX(o->max, nb_fmts-1);
> > +        min = FFMAX(o->min, -1);
> > +        max = FFMIN(o->max, nb_fmts-1);
> >      }
> 
> LGTM
> 
> maybe wait with this though after nicolas or others add av_asserts
> to check that all the min/max values make sense

Applied.
-- 
FFmpeg = Fundamentalist and Fostering Mind-dumbing Puritan Empowered Ghost


More information about the ffmpeg-devel mailing list