From e98fd8ddab98e507acce69b1bd31e2db35c8ade2 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Fri, 5 Nov 2010 12:20:21 +0100 Subject: [PATCH] Review comments for namingContexts patches --- src/man/sssd-ldap.5.xml | 12 +++--------- src/providers/ldap/sdap.c | 10 +++++----- src/providers/ldap/sdap_async.c | 18 +++++++++--------- 3 files changed, 17 insertions(+), 23 deletions(-) diff --git a/src/man/sssd-ldap.5.xml b/src/man/sssd-ldap.5.xml index 2762362..2123ffd 100644 --- a/src/man/sssd-ldap.5.xml +++ b/src/man/sssd-ldap.5.xml @@ -1056,9 +1056,7 @@ ldap_access_filter = memberOf=cn=allowedusers,ou=Groups,dc=example,dc=com Default: the value of - ldap_search_base or the value - of the defaultNamingContext or namingContexts - attribute of the RootDSE of the LDAP server. + ldap_search_base @@ -1072,9 +1070,7 @@ ldap_access_filter = memberOf=cn=allowedusers,ou=Groups,dc=example,dc=com Default: the value of - ldap_search_base or the value - of the defaultNamingContext or namingContexts - attribute of the RootDSE of the LDAP server. + ldap_search_base @@ -1088,9 +1084,7 @@ ldap_access_filter = memberOf=cn=allowedusers,ou=Groups,dc=example,dc=com Default: the value of - ldap_search_base or the value - of the defaultNamingContext or namingContexts - attribute of the RootDSE of the LDAP server. + ldap_search_base diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c index cf4e9ed..95b63d1 100644 --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -474,11 +474,11 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse, SDAP_GROUP_SEARCH_BASE, SDAP_NETGROUP_SEARCH_BASE, -1 }; - int o; + size_t c; - for (o = 0; search_base_options[o] != -1; o++) { - if (dp_opt_get_string(opts->basic, search_base_options[o]) == NULL) { + for (c = 0; search_base_options[c] != -1; c++) { + if (dp_opt_get_string(opts->basic, search_base_options[c]) == NULL) { if (naming_context == NULL) { naming_context = get_naming_context(opts->basic, rootdse); if (naming_context == NULL) { @@ -489,9 +489,9 @@ errno_t sdap_set_config_options_with_rootdse(struct sysdb_attrs *rootdse, } DEBUG(3, ("Setting option [%s] to [%s].\n", - opts->basic[search_base_options[o]].opt_name, + opts->basic[search_base_options[c]].opt_name, naming_context)); - ret = dp_opt_set_string(opts->basic, search_base_options[o], + ret = dp_opt_set_string(opts->basic, search_base_options[c], naming_context); if (ret != EOK) { DEBUG(1, ("dp_opt_set_string failed.\n")); diff --git a/src/providers/ldap/sdap_async.c b/src/providers/ldap/sdap_async.c index 01cd967..6808cea 100644 --- a/src/providers/ldap/sdap_async.c +++ b/src/providers/ldap/sdap_async.c @@ -646,16 +646,16 @@ struct tevent_req *sdap_get_rootdse_send(TALLOC_CTX *memctx, struct tevent_req *req, *subreq; struct sdap_get_rootdse_state *state; const char *attrs[] = { - "*", - "altServer", - SDAP_ROOTDSE_ATTR_NAMING_CONTEXTS, - "supportedControl", - "supportedExtension", - "supportedFeatures", - "supportedLDAPVersion", - "supportedSASLMechanisms", + "*", + "altServer", + SDAP_ROOTDSE_ATTR_NAMING_CONTEXTS, + "supportedControl", + "supportedExtension", + "supportedFeatures", + "supportedLDAPVersion", + "supportedSASLMechanisms", SDAP_ROOTDSE_ATTR_DEFAULT_NAMING_CONTEXT, - NULL + NULL }; DEBUG(9, ("Getting rootdse\n")); -- 1.7.3.2