Webrev is updated with the new bits.

On Nov 26, 2008, at 9:08 AM, Renaud Manus wrote:

> usr/src/cmd/svc/svccfg/svccfg.h
> line 253 : remove blank line

Fixed

>
>
> usr/src/cmd/svc/svccfg/svccfg_libscf.c
>
> line 1614 : you should test the return value from take_snap()

Checking the return code now...

>
> line 1914/1915 : the comment is unclear

Restated, and hopefully clarified a bit.

>
> line 1926 : uuflags is set but never used !

Fixed.

>
>
> -- Renaud
>
> Sean Wilcox wrote:
>> Bug -id : http://bugs.opensolaris.org/view_bug.do?bug_id=6765907
>> services can be started without a running snapshot
>>
>> webrev : http://cr.opensolaris.org/~swilcox/6765907/
>>
>>
> _______________________________________________
> smf-discuss mailing list
> smf-discuss at opensolaris.org
>

Sean Wilcox
x79711
303.272.9711



Reply via email to