On Wed, 27 Mar 2013 22:31:37 +0100, Diego Biurrun <[email protected]> wrote: > On Wed, Mar 13, 2013 at 08:51:55PM +0100, Anton Khirnov wrote: > > --- a/libavutil/opt.c > > +++ b/libavutil/opt.c > > @@ -564,6 +564,9 @@ static int parse_key_value_pair(void *ctx, const char > > **buf, > > char *val; > > int ret; > > > > + if (!key) > > + return AVERROR(ENOMEM); > > + > > if (*key && strspn(*buf, key_val_sep)) { > > (*buf)++; > > val = av_get_token(buf, pairs_sep); > > I wonder why you don't also check "val". >
That is a good question. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
