summaryrefslogtreecommitdiffstats
path: root/src/lib/krb5
diff options
context:
space:
mode:
authorGreg Hudson <ghudson@mit.edu>2014-05-01 18:11:32 -0400
committerGreg Hudson <ghudson@mit.edu>2014-05-07 12:58:34 -0400
commit7783054742ddd807f7b2f7157d6ed81b7fb614eb (patch)
treead18469b74a6857ad20d614186ec70d49e4157fc /src/lib/krb5
parenteba8c4909ec7ba0d7054d5d1b1061319e9970cc7 (diff)
downloadkrb5-7783054742ddd807f7b2f7157d6ed81b7fb614eb.tar.gz
krb5-7783054742ddd807f7b2f7157d6ed81b7fb614eb.tar.xz
krb5-7783054742ddd807f7b2f7157d6ed81b7fb614eb.zip
Don't remove ccache creds before storing them
Revert #6291, which attempts to suppress duplicate credentials by calling krb5_cc_remove in krb5_cc_store_cred. Most of our ccache types don't implement remove_cred, and avoiding duplicate credentials is a responsibility better suited to the ccache implementation for atomicity reasons. Removing this call gets rid of the misleading "Removing ... from ..." messages in trace logs. ticket: 7906 (new)
Diffstat (limited to 'src/lib/krb5')
-rw-r--r--src/lib/krb5/ccache/ccfns.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/lib/krb5/ccache/ccfns.c b/src/lib/krb5/ccache/ccfns.c
index 1a0bed0ac..1084d5191 100644
--- a/src/lib/krb5/ccache/ccfns.c
+++ b/src/lib/krb5/ccache/ccfns.c
@@ -84,9 +84,6 @@ krb5_cc_store_cred(krb5_context context, krb5_ccache cache,
krb5_ticket *tkt;
krb5_principal s1, s2;
- /* remove any dups */
- krb5_cc_remove_cred(context, cache, KRB5_TC_MATCH_AUTHDATA, creds);
-
TRACE_CC_STORE(context, cache, creds);
ret = cache->ops->store(context, cache, creds);
if (ret) return ret;