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


Reply via email to