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 | |
| parent | 4ddfec6b995d6673dad93cf3dd849550dde1594c (diff) | |
| download | sssd-c87a579a23b27e65ae956bc42cf0a247f2ca0baf.tar.gz sssd-c87a579a23b27e65ae956bc42cf0a247f2ca0baf.tar.xz 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')
| -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 6aeb7623f..209643a09 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; } |