Don't check mech in krb5_gss_inquire_cred_by_mech
authorGreg Hudson <ghudson@mit.edu>
Tue, 31 Jan 2012 21:35:34 +0000 (21:35 +0000)
committerGreg Hudson <ghudson@mit.edu>
Tue, 31 Jan 2012 21:35:34 +0000 (21:35 +0000)
krb5_gss_inquire_cred_by_mech checks its mech argument against two of
the four mechs a krb5 cred might have (the krb5 mech and the old krb5
mech, but not the wrong Microsoft mech or the IAKERB mech), so would
spuriously fail for the other two mechs.  There is no reason to check
the mechanism if we assume a reasonable mechglue is interpreting
application gss_inquire_cred_by_mech calls, so just remove the check.

ticket: 7084
target_version: 1.10.1
tags: pullup

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@25666 dc483132-0cff-0310-8789-dd5450dbe970

src/lib/gssapi/krb5/inq_cred.c

index 057e51bfa17b8c43725b8b071209c3e54e436ea4..78685dbaa022c73d4d29bc43b4a588ab8853f8b2 100644 (file)
@@ -224,16 +224,6 @@ krb5_gss_inquire_cred_by_mech(minor_status, cred_handle,
     OM_uint32           lifetime;
     OM_uint32           mstat;
 
-    /*
-     * We only know how to handle our own creds.
-     */
-    if ((mech_type != GSS_C_NULL_OID) &&
-        !g_OID_equal(gss_mech_krb5_old, mech_type) &&
-        !g_OID_equal(gss_mech_krb5, mech_type)) {
-        *minor_status = 0;
-        return(GSS_S_NO_CRED);
-    }
-
     cred = (krb5_gss_cred_id_t) cred_handle;
     mstat = krb5_gss_inquire_cred(minor_status,
                                   cred_handle,