Tony Nguyen wrote:
> I'd appreciate a review for
> 
> 6834760 
> <http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6834760> 
> svcadm shouldn't assert() when not enough privileges
> 
> which has been an annoyance for folks.
> 
> Webrev:
> http://cr.opensolaris.org/~tonyn/6834760/

I think the changes to configd are fine, but shouldn't there be changes to 
svcadm too?  I can't recall which scfdie() we actually hit, but if the 
user doesn't have privs, we still shouldn't assert(), we should give an 
appropriate permission error.

liane

Reply via email to