diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2012-10-09 19:02:05 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-10-12 09:56:54 +0200 |
| commit | 115cc768599d7df4b3206426652d3e7a3971d597 (patch) | |
| tree | a3d814c10ce9fc69f1ec0df90712f6968ab663ee /src/providers/ldap/ldap_child.c | |
| parent | fc1a05ca545806ff784921c6751129c3d93d27af (diff) | |
| download | sssd-115cc768599d7df4b3206426652d3e7a3971d597.tar.gz sssd-115cc768599d7df4b3206426652d3e7a3971d597.tar.xz sssd-115cc768599d7df4b3206426652d3e7a3971d597.zip | |
Two fixes to child processes
There was an unused structure member in the krb5_child.
Declaration of __krb5_error_msg was shadowing the same variable from
sss_krb5.h which is not nice. Also we might actually use the error
context directly instead of passing it as parameter.
Diffstat (limited to 'src/providers/ldap/ldap_child.c')
| -rw-r--r-- | src/providers/ldap/ldap_child.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/providers/ldap/ldap_child.c b/src/providers/ldap/ldap_child.c index 888a1249f..35e0c0f5b 100644 --- a/src/providers/ldap/ldap_child.c +++ b/src/providers/ldap/ldap_child.c @@ -35,12 +35,13 @@ #include "providers/dp_backend.h" static krb5_context krb5_error_ctx; +#define LDAP_CHILD_DEBUG(level, error) KRB5_DEBUG(level, krb5_error_ctx, error) -static const char *__krb5_error_msg; +static const char *__ldap_child_krb5_error_msg; #define KRB5_SYSLOG(krb5_error) do { \ - __krb5_error_msg = sss_krb5_get_error_message(krb5_error_ctx, krb5_error); \ - sss_log(SSS_LOG_ERR, "%s", __krb5_error_msg); \ - sss_krb5_free_error_message(krb5_error_ctx, __krb5_error_msg); \ + __ldap_child_krb5_error_msg = sss_krb5_get_error_message(krb5_error_ctx, krb5_error); \ + sss_log(SSS_LOG_ERR, "%s", __ldap_child_krb5_error_msg); \ + sss_krb5_free_error_message(krb5_error_ctx, __ldap_child_krb5_error_msg); \ } while(0) struct input_buffer { |