On Tue, Aug 13, 2002 at 11:49:56AM -0400, Alan DeKok wrote:
> Simon <[EMAIL PROTECTED]> wrote:
> > +   if (!argv[1])
> > +           usage();
> > +   if (!strlen(argv[1]))
> > +           usage();
> 
>   That's *very* bad, as it accesses memory which may not exist.  It
> would be better to check 'argc' against 'optind', which doesn't access
> non-existent elements of arrays.

Blah, yeah, sorry, you're obviously correct. It's to hot here to think
right now.

-- 
Simon


- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to