diff options
author | Greg Hudson <ghudson@mit.edu> | 2014-05-01 18:11:32 -0400 |
---|---|---|
committer | Greg Hudson <ghudson@mit.edu> | 2014-05-07 12:58:34 -0400 |
commit | 7783054742ddd807f7b2f7157d6ed81b7fb614eb (patch) | |
tree | ad18469b74a6857ad20d614186ec70d49e4157fc /src/lib/krb5 | |
parent | eba8c4909ec7ba0d7054d5d1b1061319e9970cc7 (diff) | |
download | krb5-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.c | 3 |
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; |