ldap/servers/slapd/attrsyntax.c | 20 +++++++++++++++++++-
1 file changed, 19 insertions(+), 1 deletion(-)
New commits:
commit 2b7cbb8dfe6725f0ce3a19707dea8b8c59c6ea46
Author: Nathan Kinder <nkinder(a)redhat.com>
Date: Tue Oct 22 14:28:34 2013 -0700
Ticket 47569 - ACIs do not allow attribute subtypes in targetattr keyword
When validating the targetattr ACI keyword, we check if the attribute
is defined in the schema. This schema check fails if the attribute
has a subtype present.
This patch makes the attribute syntax lookup function check if a
subtype was specified before performing the looking. If a subtype
is specified, it is stripped off and we just use the base attribute
name to lookup the syntax from the hashtable.
(cherry picked from commit cb73cf2b09b015696d0ef04820cfbbb564143207)
diff --git a/ldap/servers/slapd/attrsyntax.c b/ldap/servers/slapd/attrsyntax.c
index 7abd6b7..a08ea96 100644
--- a/ldap/servers/slapd/attrsyntax.c
+++ b/ldap/servers/slapd/attrsyntax.c
@@ -537,10 +537,28 @@ int
attr_syntax_exists(const char *attr_name)
{
struct asyntaxinfo *asi;
+ char *check_attr_name = NULL;
+ char *p = NULL;
+ int free_attr = 0;
+
+ /* Ignore any attribute subtypes. */
+ if (p = strchr(attr_name, ';')) {
+ int check_attr_len = p - attr_name + 1;
+
+ check_attr_name = (char *)slapi_ch_malloc(check_attr_len);
+ PR_snprintf(check_attr_name, check_attr_len, "%s", attr_name);
+ free_attr = 1;
+ } else {
+ check_attr_name = attr_name;
+ }
- asi = attr_syntax_get_by_name(attr_name);
+ asi = attr_syntax_get_by_name(check_attr_name);
attr_syntax_return( asi );
+ if (free_attr) {
+ slapi_ch_free_string(&check_attr_name);
+ }
+
if ( asi != NULL )
{
return 1;