[389-commits] Branch 'Directory_Server_8_2_Branch' - ldap/servers

[Noriko Hosoi]

 ldap/servers/plugins/acl/aclparse.c |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit 14fc7fca2a92566e483382492527c2991b92d88c
Author: Noriko Hosoi <nhosoi at redhat.com>
Date:   Fri May 21 18:26:53 2010 -0700

    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.

diff --git a/ldap/servers/plugins/acl/aclparse.c b/ldap/servers/plugins/acl/aclparse.c
index b128ff8..8c88bf4 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] == '"') {