From 21895a2f9676980e6cdcfa7d8ad963cfbfbe8724 Mon Sep 17 00:00:00 2001 From: Stephane Bortzmeyer Date: Fri, 25 Jan 2008 13:53:50 +0000 Subject: [PATCH] #1879652 Implements the (objectClass=dummystuff) solution --- SRC/plugins/ldap/ldap.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/SRC/plugins/ldap/ldap.c b/SRC/plugins/ldap/ldap.c index adbec53..dcd52a9 100644 --- a/SRC/plugins/ldap/ldap.c +++ b/SRC/plugins/ldap/ldap.c @@ -122,13 +122,13 @@ start() * depreciated in a later release. * * So, we perform a dummy search immediately. + * + * See #1879652 for why we use "dummystuff" and not "*" + * */ - /* TODO: Since it is just to see if the server replies, we could use a - * request like (objectclass=dummystuff) to be sure to not return anything */ - result = ldap_search_s(session, base, LDAP_SCOPE_ONELEVEL, "(objectclass=*)", NULL, /* Return - * all - * attributes - */ + result = ldap_search_s(session, base, LDAP_SCOPE_ONELEVEL, "(objectClass=People)", NULL, /* Return + * * all * + * attributes */ 1, &response); if (result != 0) { err_quit