r21561: It makes absolutely no sense to call krb5_kt_resolve() two times

directly after another.

Guenther
(This used to be commit 76ba11d7770bac7c6db2eb1640139bbe270d82c3)

diff --git a/source3/libads/kerberos_keytab.c b/source3/libads/kerberos_keytab.c
index 4b640151d3dc0d9158e09eaa919ae9c9b22398fd..ba1a91656742c67766a813b2583f346d6c9dce5e 100644 (file)
--- a/source3/libads/kerberos_keytab.c
+++ b/source3/libads/kerberos_keytab.c
@@ -408,12 +408,7 @@ int ads_keytab_flush(ADS_STRUCT *ads)
        DEBUG(3,("ads_keytab_flush: Using default keytab: %s\n", (char *) &keytab_name));
        ret = krb5_kt_resolve(context, (char *) &keytab_name, &keytab);
        if (ret) {
-               DEBUG(1,("ads_keytab_flush: krb5_kt_default failed (%s)\n", error_message(ret)));
-               goto out;
-       }
-       ret = krb5_kt_resolve(context, (char *) &keytab_name, &keytab);
-       if (ret) {
-               DEBUG(1,("ads_keytab_flush: krb5_kt_default failed (%s)\n", error_message(ret)));
+               DEBUG(1,("ads_keytab_flush: krb5_kt_resolve failed (%s)\n", error_message(ret)));
                goto out;
        }