ntdll: remove an unnecessary NULL check : Coverity
lats at yless4u.com.au
Tue Jan 2 00:28:38 CST 2007
Dmitry Timoshkov wrote:
> "Jeff Latimer" <lats at yless4u.com.au> wrote:
>> Coverity Cid:344 highlighted a REVERSE-INULL. This patch removes the
>> redundant null tests as it is clear from the code above that the
>> pointer has already been used to dereference and can't be null.
> Yes, 'attr' is used without a NULL check earlier, but that's an actual
> bug, and that's where the code needs to be fixed.
So in this case my change is ok for the tidy up that it is, it just that
there is another problem. Is this in bugzilla?
More information about the wine-devel