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:22:00 -0700 |
| commit | d402f62d438a72804cff14315bcac14dfdb60675 (patch) | |
| tree | ab0babc245b6b7a2684ee82a869769947d115d17 /src/providers/krb5 | |
| parent | 5ee1287ca6c4105a3f659d62dbd8064c7c280bf8 (diff) | |
| download | sssd-d402f62d438a72804cff14315bcac14dfdb60675.tar.gz sssd-d402f62d438a72804cff14315bcac14dfdb60675.tar.xz sssd-d402f62d438a72804cff14315bcac14dfdb60675.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; } |