summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2012-02-24 15:46:29 -0500
committerStephen Gallagher <sgallagh@redhat.com>2012-02-26 20:36:48 -0500
commitf3dc9ac52a808fcbedc7fb1ae83964b523c72df2 (patch)
tree35058e0944c5c1c7ccaeb471e2bed1f2ac95b02b
parente6ab4b3ded22aca8dd6eb3ce69630d931cddb140 (diff)
downloadsssd-f3dc9ac52a808fcbedc7fb1ae83964b523c72df2.tar.gz
sssd-f3dc9ac52a808fcbedc7fb1ae83964b523c72df2.tar.xz
sssd-f3dc9ac52a808fcbedc7fb1ae83964b523c72df2.zip
LDAP: Remove unnecessary filter sanitize
The orig_dn here isn't being passed to a filter and therefore must not be santized, as the sanitization process would break DNs that contain (among other things) parentheses.
-rw-r--r--src/providers/ldap/sdap_async_autofs.c16
1 files changed, 5 insertions, 11 deletions
diff --git a/src/providers/ldap/sdap_async_autofs.c b/src/providers/ldap/sdap_async_autofs.c
index ea034c2f5..528e8b73a 100644
--- a/src/providers/ldap/sdap_async_autofs.c
+++ b/src/providers/ldap/sdap_async_autofs.c
@@ -185,7 +185,7 @@ struct automntmaps_process_members_state {
int timeout;
struct sysdb_ctx *sysdb;
- char *clean_orig_dn;
+ const char *orig_dn;
char *base_filter;
char *filter;
const char **attrs;
@@ -217,7 +217,6 @@ automntmaps_process_members_send(TALLOC_CTX *mem_ctx,
errno_t ret;
struct tevent_req *req;
struct automntmaps_process_members_state *state;
- const char *orig_dn;
req = tevent_req_create(mem_ctx, &state,
struct automntmaps_process_members_state);
@@ -250,20 +249,15 @@ automntmaps_process_members_send(TALLOC_CTX *mem_ctx,
goto immediate;
}
- ret = sysdb_attrs_get_string(state->map, SYSDB_ORIG_DN, &orig_dn);
- if (ret != EOK) {
- DEBUG(SSSDBG_CRIT_FAILURE, ("Cannot get originalDN\n"));
- goto immediate;
- }
- ret = sss_filter_sanitize(state, orig_dn, &state->clean_orig_dn);
+ ret = sysdb_attrs_get_string(state->map, SYSDB_ORIG_DN, &state->orig_dn);
if (ret != EOK) {
- DEBUG(SSSDBG_CRIT_FAILURE, ("Cannot sanitize originalDN\n"));
+ DEBUG(SSSDBG_CRIT_FAILURE, ("Cannot get originalDN\n"));
goto immediate;
}
DEBUG(SSSDBG_TRACE_FUNC,
- ("Examining autofs map [%s]\n", state->clean_orig_dn));
+ ("Examining autofs map [%s]\n", state->orig_dn));
ret = automntmaps_process_members_next_base(req);
if (ret != EOK) {
@@ -304,7 +298,7 @@ automntmaps_process_members_next_base(struct tevent_req *req)
state->search_bases[state->base_iter]->basedn));
subreq = sdap_get_generic_send(state, state->ev, state->opts, state->sh,
- state->clean_orig_dn,
+ state->orig_dn,
state->search_bases[state->base_iter]->scope,
state->filter, state->attrs,
state->opts->autofs_entry_map,