This looks good to me. It probably could have been coded differently using getopt, but since that wasn't used in the original code, I think your changes are acceptable.
--Glenn Bart Blanquart wrote: > > Please review my changes for > > PSARC 2007/369 auths(1) update: > command needs option for scripting > > The webrev is available at http://cr.opensolaris.org/~bartbl/6251549/. > > The change is fairly straightforward: the addition of an option to > auths(1), permitting checking if an authorization has been granted: > > SYNOPSIS > auths [ user ...] > auths -a authorization [ user ] > > (...) > OPTIONS > The following option is supported: > > -a authorization > > Does not write anything to the standard output. Exits with > zero status if you or the optionally specified user has > been granted this authorization. > > I've tested a series of combinations of arguments to the command; the > script I've used and its output can be found at > http://cr.opensolaris.org/~bartbl/6251549-testing/. > > Bart > _______________________________________________ > security-discuss mailing list > security-discuss at opensolaris.org