dlls/advapi32/tests/security.c - Bugfix in test!

Alexandre Julliard julliard at winehq.org
Fri Mar 18 07:44:47 CST 2005


Jakob Eriksson <jakov at vmlinux.org> writes:

> --- dlls/advapi32/tests/security.c	14 Mar 2005 17:20:58 -0000	1.12
> +++ dlls/advapi32/tests/security.c	16 Mar 2005 09:32:28 -0000
> @@ -289,8 +289,8 @@
>          luid.LowPart = i;
>          cchName = sizeof(buf);
>          ret = pLookupPrivilegeNameA(NULL, &luid, buf, &cchName);
> -        ok( ret && GetLastError() != ERROR_NO_SUCH_PRIVILEGE,
> -         "LookupPrivilegeNameA(0.%ld) failed: %ld\n", i, GetLastError());
> +	if (GetLastError() != ERROR_NO_SUCH_PRIVILEGE)
> +            ok( ret, "LookupPrivilegeNameA(0.%ld) failed: %ld\n", i, GetLastError());

It doesn't really make sense to check the last error if the function
succeeded.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list