krb5 commit [krb5-1.14]: Skip unnecessary mech calls in gss_inquire_cred()

Tom Yu tlyu at mit.edu
Wed Apr 13 15:13:40 EDT 2016


https://github.com/krb5/krb5/commit/1f3e550f5c7a626d45c8bacccb6d52079308aa7e
commit 1f3e550f5c7a626d45c8bacccb6d52079308aa7e
Author: Greg Hudson <ghudson at mit.edu>
Date:   Mon Feb 29 16:51:22 2016 -0500

    Skip unnecessary mech calls in gss_inquire_cred()
    
    If the caller does not request a name, lifetime, or cred_usage when
    calling gss_inquire_cred(), service the call by copying the mechanism
    list (if requested) but do not call into the mech.
    
    This change alleviates an issue (reported by Adam Bernstein) where
    SPNEGO can fail in the presence of expired krb5 credentials rather
    than proceeding with a different mechanism, or can resolve a krb5
    credential without the benefit of the target name.
    
    (cherry picked from commit ff5eb892910eeac335d989ae14020da4ffbcc8ec)
    
    ticket: 8373
    version_fixed: 1.14.2

 src/lib/gssapi/mechglue/g_inq_cred.c |   41 +++++++++++++++++++---------------
 1 files changed, 23 insertions(+), 18 deletions(-)

diff --git a/src/lib/gssapi/mechglue/g_inq_cred.c b/src/lib/gssapi/mechglue/g_inq_cred.c
index c5577d4..9111962 100644
--- a/src/lib/gssapi/mechglue/g_inq_cred.c
+++ b/src/lib/gssapi/mechglue/g_inq_cred.c
@@ -92,27 +92,32 @@ gss_OID_set *		mechanisms;
 	mech_cred = GSS_C_NO_CREDENTIAL;
 	mech = gssint_get_mechanism(GSS_C_NULL_OID);
     }
-    if (mech == NULL)
-	return (GSS_S_DEFECTIVE_CREDENTIAL);
-    if (!mech->gss_inquire_cred)
-	return (GSS_S_UNAVAILABLE);
 
-    status = mech->gss_inquire_cred(minor_status, mech_cred,
-				    name ? &mech_name : NULL,
-				    lifetime, cred_usage, NULL);
-    if (status != GSS_S_COMPLETE) {
-	map_error(minor_status, mech);
-	return(status);
-    }
+    /* Skip the call into the mech if the caller doesn't care about any of the
+     * values we would ask for. */
+    if (name != NULL || lifetime != NULL || cred_usage != NULL) {
+	if (mech == NULL)
+	    return (GSS_S_DEFECTIVE_CREDENTIAL);
+	if (!mech->gss_inquire_cred)
+	    return (GSS_S_UNAVAILABLE);
 
-    if (name) {
-	/* Convert mech_name into a union_name equivalent. */
-	status = gssint_convert_name_to_union_name(&temp_minor_status,
-						   mech, mech_name, name);
+	status = mech->gss_inquire_cred(minor_status, mech_cred,
+					name ? &mech_name : NULL,
+					lifetime, cred_usage, NULL);
 	if (status != GSS_S_COMPLETE) {
-	    *minor_status = temp_minor_status;
 	    map_error(minor_status, mech);
-	    return (status);
+	    return(status);
+	}
+
+	if (name) {
+	    /* Convert mech_name into a union_name equivalent. */
+	    status = gssint_convert_name_to_union_name(&temp_minor_status,
+						       mech, mech_name, name);
+	    if (status != GSS_S_COMPLETE) {
+		*minor_status = temp_minor_status;
+		map_error(minor_status, mech);
+		return (status);
+	    }
 	}
     }
 


More information about the cvs-krb5 mailing list