kernel32: add a test for CreateFileA()

Nikolay Sivov nsivov at codeweavers.com
Sun Jul 18 01:24:29 CDT 2010


  On 7/18/2010 10:17, Austin English wrote:
> Passed all the vm's on wtb.
Hi, Austin. This looks strange:

> +
> +    hFile = CreateFileA("c:\\*.*", GENERIC_READ,FILE_SHARE_READ,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL );
> +    ok(GetLastError() == ERROR_INVALID_NAME || broken(GetLastError() == ERROR_FILE_NOT_FOUND) || broken(GetLastError() == ERROR_PATH_NOT_FOUND),
> +       "CreateFileA should have returned ERROR_INVALID_NAME on %s, but got %u\n", filename, GetLastError());
> +    CloseHandle( hFile );
If a call failed then you should test for null hFile I suppose and 
remove CloseHandle().  Also when testing for last error a common rule is 
to set it first before the call.

Does anything depend on this having 3 possible return values?



More information about the wine-devel mailing list