ntdll: remove an unnecessary NULL check : Coverity

Dmitry Timoshkov dmitry at codeweavers.com
Tue Jan 2 00:02:14 CST 2007


"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.

-- 
Dmitry.



More information about the wine-devel mailing list