Hi Ginnie.

The bug report says that the issue is the quotes around the list of groups, but the bugfix is more than that. Looks to me that rather than get the list of groups from svccfg, it assumes the whole set (the SC_VALID_GROUPS list). This indeed seems like a better check.

I don't understand the message at 519 though: SC_VALID_GROUPS includes many groups other than "system", as defined by lines 76-81. How come the message says "system" is the only valid group? Also, the message will be printed potentially many times as 520 is called from a loop.

Also, I notice the suggested fix in the bug report is out of date, but maybe that's because you've iterated on this fix. (I haven't been paying attention.)

    Thanks,
    Jack


 but can you please update the evaluation with this info.



On 09/ 7/11 08:11 AM, Virginia Wray wrote:
Hi --

This is a stopper bug that I would like to get back quickly, so
that we can redeliver for build 174.
Jan has code reviewed. Can I get one more reviewer
and Dave's ok?

CR:
http://monaco.us.oracle.com/detail.jsf?cr=7087609

Webrev:
file:///net/indiana-build.us.oracle.com/data/share/vw130254/projectwork/sys-unconfig/7087609/slim_source/webrev/index.html


Thanks,
ginnie
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to