secur32/tests: Skip the tests instead of crashing if Kerberos is not supported.

This avoids a crash when Wine is compiled using --without-krb5. This makes test_ticket_cache() more consistent with test_kerberos().

secur32/tests: Skip the tests instead of crashing if Kerberos is not supported.
db89311a · Francois Gouget · Alexandre Julliard · 8f7483d0 · db89311a
Commit db89311a authored Oct 09, 2023 by Francois Gouget Committed by Alexandre Julliard Oct 13, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 2 deletions

secur32.c dlls/secur32/tests/secur32.c +7 -2

No files found.
--- a/dlls/secur32/tests/secur32.c
+++ b/dlls/secur32/tests/secur32.c
@@ -446,7 +446,7 @@ static void test_kerberos(void)


    status = QuerySecurityPackageInfoA(provider, &info);
-    ok(status == SEC_E_OK, "Kerberos package not installed, skipping test\n");
+    ok(status == SEC_E_OK, "Kerberos package not installed (%08lx), skipping test\n", status);
    if(status != SEC_E_OK)
        return;

@@ -479,7 +479,12 @@ static void test_ticket_cache(void)

    RtlInitAnsiString( &name, MICROSOFT_KERBEROS_NAME_A );
    status = LsaLookupAuthenticationPackage( lsa, &name, &package );
-    ok( !status, "got %08lx\n", status );
+    ok(status == SEC_E_OK, "Kerberos package not installed (%08lx), skipping test\n", status);
+    if(status != SEC_E_OK)
+    {
+      LsaDeregisterLogonProcess( lsa );
+      return;
+    }

    status = LsaCallAuthenticationPackage( lsa, package, &req, sizeof(req), (void **)&resp, &len, &status );
    ok( !status, "got %08lx\n", status );