Attachment 231501 Details for Bug 232910 – diffs

[patch] diffs

cvsdiffs (text/plain), 1.08 KB, created by Rich Megginson on 2007-10-18 20:09:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rich Megginson

Created: 2007-10-18 20:09:54 UTC

Size: 1.08 KB

patch

obsolete

>Index: ldapserver/ldap/servers/plugins/acl/aclparse.c
>===================================================================
>RCS file: /cvs/dirsec/ldapserver/ldap/servers/plugins/acl/aclparse.c,v
>retrieving revision 1.7
>diff -u -8 -r1.7 aclparse.c
>--- ldapserver/ldap/servers/plugins/acl/aclparse.c	18 Oct 2007 00:08:27 -0000	1.7
>+++ ldapserver/ldap/servers/plugins/acl/aclparse.c	18 Oct 2007 20:09:01 -0000
>@@ -1227,20 +1227,26 @@
> 	Targetattr	**attrArray;
> 	char		*s, *end_attr, *str;
> 	int		len;
> 	Targetattr	*attr = NULL;
> 
> 	s = strchr (attr_val, '=');
> 	s++;
> 	__acl_strip_leading_space(&s);
>+	__acl_strip_trailing_space(s);
> 	len = strlen(s);
> 	if (*s == '"' && s[len-1] == '"') {
> 		s[len-1] = '\0';
> 		s++;
>+	} else {
>+		slapi_log_error(SLAPI_LOG_FATAL, plugin_name,
>+						"__aclp__init_targetattr: WARNING: The targetattr clause does not begin and end with a \": [%s]\n",
>+						s);
>+		/* tempted to return ACL_SYNTAX_ERR here, but what would that break? */
> 	}
> 
> 	str = s;
> 	attrArray = aci->targetAttr;
> 
> 	if (attrArray[0] != NULL) {
> 		/* 
> 		** That means we are visiting more than once.

Actions: View | Diff

Attachments on bug 232910: 231501 | 231531 | 231601