diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-05-21 18:26:53 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-05-21 18:26:53 -0700 |
| commit | 08a38d39590e4117673ca5d5fe2f4d6b1924da1f (patch) | |
| tree | 963b7e2516f86db615110d6c749e8f649d8cb073 | |
| parent | 80f3188a3a9789824128f1f6baede66de8c92c37 (diff) | |
| download | ds-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.
| -rw-r--r-- | ldap/servers/plugins/acl/aclparse.c | 1 |
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] == '"') { |