Don't compare file handles to NULL

Michael Stefaniuc mstefani at redhat.com
Thu Mar 8 07:48:34 CST 2007


Alexandre Julliard wrote:
> Francois Gouget <fgouget at free.fr> writes:
> 
> 
>>These are not false positives. Any file handle that is not 
>>INVALID_HANDLE_VALUE must be closed with CloseHandle(). So these checks 
>>should be against INVALID_HANDLE_VALUE, not NULL. In fact they may 
>>possibly be removed altogether.
> 
> 
> Note that a valid file handle will never be NULL, so while these
> checks are wrong in theory, in practice it makes no difference.
Well, in one of the found cases (one of the RPCRT) the code was setting
the file handle manualy to NULL...

bye
	michael
-- 
Michael Stefaniuc               Tel.: +49-711-96437-199
Sr. Network Engineer            Fax.: +49-711-96437-111
Red Hat GmbH                    Email: mstefani at redhat.com
Hauptstaetterstr. 58            http://www.redhat.de/
D-70178 Stuttgart



More information about the wine-devel mailing list