line 1957 - 1958 : unneeded since we aborting in bad_error()

line 2046 : 0 is fine but since it's a pointer,
             NULL would be more appropriate.

-- Renaud

Sean Wilcox wrote:
> Webrev updated with new fix.
> 
> http://cr.opensolaris.org/~swilcox/6765907/
> 
> Bug -id : http://bugs.opensolaris.org/view_bug.do?bug_id=6765907
>  services can be started without a running snapshot
> 
> 
> Sean Wilcox wrote:
>> Tom's got a good point here, even using uu_list_insert_before(list,  
>> uu_list_first(list), "enabled element") I have no guarantee that this  
>> element will stay the first element, and that I can keep other things  
>> from dumping in front of it on the list.
>>
>> So back to the drawing board, with a shot a Tom's idea here... I'll  
>> repost the webrev when complete.
>>
>>
>>
>>   

Reply via email to