diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2012-01-20 10:40:04 -0500 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-01-23 08:19:11 -0500 |
| commit | 355fe74d2eaab2e021d75437a6b997fd24b9c4e5 (patch) | |
| tree | 0da883b5bd03f9359f200d5fff23804ea52d126c /src/providers | |
| parent | 9d1d6019a27a0ed70a665700a9f6acc55f874811 (diff) | |
| download | sssd-355fe74d2eaab2e021d75437a6b997fd24b9c4e5.tar.gz sssd-355fe74d2eaab2e021d75437a6b997fd24b9c4e5.tar.xz sssd-355fe74d2eaab2e021d75437a6b997fd24b9c4e5.zip | |
LDAP: Improve debugging for sdap_parse_deref
Move the debug statement identifying the DN to an earlier line, so
if we get a reply with no attributes, we know which entry is at
fault.
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ldap/sdap.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c index 3ca2e2861..c366dda0b 100644 --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -323,15 +323,18 @@ errno_t sdap_parse_deref(TALLOC_CTX *mem_ctx, goto done; } + orig_dn = dref->derefVal.bv_val; + DEBUG(SSSDBG_TRACE_LIBS, + ("Dereferenced DN: %s\n", orig_dn)); + if (!dref->attrVals) { - DEBUG(2, ("Dereferenced entry has no attributes\n")); + DEBUG(SSSDBG_MINOR_FAILURE, + ("Dereferenced entry [%s] has no attributes\n", + orig_dn)); ret = EINVAL; goto done; } - orig_dn = dref->derefVal.bv_val; - DEBUG(7, ("Dereferenced DN: %s\n", orig_dn)); - ocs = NULL; for (dval = dref->attrVals; dval != NULL; dval = dval->next) { if (strcasecmp("objectClass", dval->type) == 0) { |