I fully support the changes described here, but I have a couple of 
comments/questions about minor details.

1. The man page should document the specific authorizations needed, just 
as it used to document the specific privilege. It's fine to mention that 
the Maintenance and Repair profile provides these authorizations, but 
they could be provided by other profiles also. [In other words, the 
authorization name is the real interface, not the profile name.]

>                       Multiple -e and -d options can  be  specified  on
> -                     the  command  line. Only users with the sys_admin
> -                     privilege can use this option.
> +                     the  command  line. Only users and roles belonging
> +                  to the "Maintenance and Repair" RBAC profile can
> +                  use this option.

Suggest: Only users and roles with the solaris.smf.manage.coreadm and 
solaris.smf.value.coreadm authorizations can use this option.


2. The text below may imply more than you intended:

> +     /var/cores
> +
> +         Default depository for core files.
> +
> +

It appears that this case creates the /var/cores directory but doesn't 
change the default core file pattern to put core files there. Unless 
I've overlooked that part of the proposal, the wording used at the end 
of the man page is probably better: "Directory provided for global core 
file storage"

        Scott

Reply via email to