source-changes: Re: CVS commit: src/sys/ufs/lfs

Subject: Re: CVS commit: src/sys/ufs/lfs
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Tyler Retzlaff <rtr@omicron-persei-8.net>
List: source-changes
Date: 03/19/2006 18:47:16

YAMAMOTO Takashi wrote:

>>Module Name:	src
>>Committed By:	rtr
>>Date:		Sun Mar 19 04:10:03 UTC 2006
>>
>>Modified Files:
>>	src/sys/ufs/lfs: lfs_syscalls.c
>>
>>Log Message:
>>init struct vnode *vp = NULL
>>coverity 2724 / run 6
>>XXX in future runs coverity may complain about deref NULL now but comment
>>    on line 382 indicates this should not be possible
>>    
>>
>
>is it a real fix, or just to appease coverity?
>  
>
It is a real fix.  The XXX is just an indicator that coverity will 
probably complain about
NULL deref now which could be avoided but then that would only be for 
the purpose of appeasing coverity.

>YAMAMOTO Takashi
>  
>