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

[Jakob Eriksson]

Alexandre Julliard wrote:

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

True. Don't know what I was thinking.


regards,
Jakob