summaryrefslogtreecommitdiff
path: root/src/providers/ldap
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2012-10-09 19:02:05 +0200
committerJakub Hrozek <jhrozek@redhat.com>2012-10-12 09:56:54 +0200
commit115cc768599d7df4b3206426652d3e7a3971d597 (patch)
treea3d814c10ce9fc69f1ec0df90712f6968ab663ee /src/providers/ldap
parentfc1a05ca545806ff784921c6751129c3d93d27af (diff)
downloadsssd-115cc768599d7df4b3206426652d3e7a3971d597.tar.gz
sssd-115cc768599d7df4b3206426652d3e7a3971d597.tar.bz2
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')
-rw-r--r--src/providers/ldap/ldap_child.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/src/providers/ldap/ldap_child.c b/src/providers/ldap/ldap_child.c
index 888a1249..35e0c0f5 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 {