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/ldap/ldap_child.c | |
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/ldap/ldap_child.c')
-rw-r--r-- | src/providers/ldap/ldap_child.c | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/src/providers/ldap/ldap_child.c b/src/providers/ldap/ldap_child.c index 025236e5e..e66406c0e 100644 --- a/src/providers/ldap/ldap_child.c +++ b/src/providers/ldap/ldap_child.c @@ -228,8 +228,10 @@ static krb5_error_code ldap_child_get_tgt_sync(TALLOC_CTX *memctx, krberr = krb5_kt_default(context, &keytab); } if (krberr) { - DEBUG(0, ("Failed to read keytab file: %s\n", - sss_krb5_get_error_message(context, krberr))); + DEBUG(SSSDBG_FATAL_FAILURE, + ("Failed to read keytab file [%s]: %s\n", + KEYTAB_CLEAN_NAME, + sss_krb5_get_error_message(context, krberr))); goto done; } @@ -272,11 +274,14 @@ static krb5_error_code ldap_child_get_tgt_sync(TALLOC_CTX *memctx, keytab, 0, NULL, &options); if (krberr) { - DEBUG(0, ("Failed to init credentials: %s\n", - sss_krb5_get_error_message(context, krberr))); - sss_log(SSS_LOG_ERR, "Failed to initialize credentials using keytab [%s]: %s. " - "Unable to create GSSAPI-encrypted LDAP connection.", - keytab_name, sss_krb5_get_error_message(context, krberr)); + DEBUG(SSSDBG_FATAL_FAILURE, + ("Failed to init credentials: %s\n", + sss_krb5_get_error_message(context, krberr))); + sss_log(SSS_LOG_ERR, + "Failed to initialize credentials using keytab [%s]: %s. " + "Unable to create GSSAPI-encrypted LDAP connection.", + KEYTAB_CLEAN_NAME, + sss_krb5_get_error_message(context, krberr)); goto done; } |