summaryrefslogtreecommitdiffstats
path: root/ldap/servers/plugins/acl
diff options
context:
space:
mode:
authorNoriko Hosoi <nhosoi@redhat.com>2010-05-21 18:26:53 -0700
committerNoriko Hosoi <nhosoi@redhat.com>2010-05-21 18:26:53 -0700
commit08a38d39590e4117673ca5d5fe2f4d6b1924da1f (patch)
tree963b7e2516f86db615110d6c749e8f649d8cb073 /ldap/servers/plugins/acl
parent80f3188a3a9789824128f1f6baede66de8c92c37 (diff)
downloadds-08a38d39590e4117673ca5d5fe2f4d6b1924da1f.tar.gz
ds-08a38d39590e4117673ca5d5fe2f4d6b1924da1f.tar.xz
ds-08a38d39590e4117673ca5d5fe2f4d6b1924da1f.zip
593899 - adding specific ACI causes very large mem allocate request
https://bugzilla.redhat.com/show_bug.cgi?id=593899 Additional fix: if a target value is double quoted followed by trailing spaces, the double quotes were not correctly handled. Sample failed case: aci: ( target = "ldap:///ou=organizationalUnit, o=test_ACIs.com" ) ^ Changed to call __acl_strip_trailing_space to remove them.
Diffstat (limited to 'ldap/servers/plugins/acl')
-rw-r--r--ldap/servers/plugins/acl/aclparse.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/ldap/servers/plugins/acl/aclparse.c b/ldap/servers/plugins/acl/aclparse.c
index b128ff89..8c88bf4a 100644
--- a/ldap/servers/plugins/acl/aclparse.c
+++ b/ldap/servers/plugins/acl/aclparse.c
@@ -326,6 +326,7 @@ __aclp__parse_aci (char *str, aci_t *aci_item)
if ( (s = strchr( str, '=' )) != NULL ) {
value = s + 1;
__acl_strip_leading_space(&value);
+ __acl_strip_trailing_space(value);
len = strlen ( value );
/* strip double quotes */
if (*value == '"' && value[len-1] == '"') {