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