diff options
author | Stephen Gallagher <sgallagh@redhat.com> | 2012-04-04 06:33:11 -0700 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-04-05 15:21:34 -0700 |
commit | c87a579a23b27e65ae956bc42cf0a247f2ca0baf (patch) | |
tree | 71756a8a898322d4e55d96e820e07d9ab5d55fce /src/providers/krb5/krb5_child.c | |
parent | 4ddfec6b995d6673dad93cf3dd849550dde1594c (diff) | |
download | sssd-c87a579a23b27e65ae956bc42cf0a247f2ca0baf.tar.gz sssd-c87a579a23b27e65ae956bc42cf0a247f2ca0baf.tar.bz2 sssd-c87a579a23b27e65ae956bc42cf0a247f2ca0baf.zip |
Clean up log messages about keytab_name
There were many places where we were printing (null) to the logs
because a NULL keytab name tells libkrb5 to use its configured
default instead of a particular path. This patch should clean up
all uses of this to print "default" in the logs.
https://fedorahosted.org/sssd/ticket/1288
Diffstat (limited to 'src/providers/krb5/krb5_child.c')
-rw-r--r-- | src/providers/krb5/krb5_child.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/providers/krb5/krb5_child.c b/src/providers/krb5/krb5_child.c index 6aeb7623..209643a0 100644 --- a/src/providers/krb5/krb5_child.c +++ b/src/providers/krb5/krb5_child.c @@ -1273,8 +1273,10 @@ static krb5_error_code check_fast_ccache(krb5_context ctx, const char *primary, kerr = krb5_kt_default(ctx, &keytab); } if (kerr) { - DEBUG(0, ("Failed to read keytab file [%s].\n", - keytab_name != NULL ? keytab_name : "(default)")); + DEBUG(SSSDBG_FATAL_FAILURE, + ("Failed to read keytab file [%s]: %s\n", + KEYTAB_CLEAN_NAME, + sss_krb5_get_error_message(ctx, kerr))); goto done; } |