]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/aclparse.c
reject registrations when back-monitor is not configured
[openldap] / servers / slapd / aclparse.c
index 5a7184a25ee60a748f6f5445f66d59d546f920e7..258917a639080e0ce031374d6b72f6f30c72a29c 100644 (file)
 #include "lber_pvt.h"
 #include "lutil.h"
 
-static char *style_strings[] = {
+static const char style_base[] = "base";
+char *style_strings[] = {
        "regex",
        "expand",
-       "base",
+       "exact",
        "one",
        "subtree",
        "children",
@@ -69,7 +70,14 @@ static int           check_scope( BackendDB *be, AccessControl *a );
 
 #ifdef SLAP_DYNACL
 static int
-slap_dynacl_config( const char *fname, int lineno, Access *b, const char *name, slap_style_t sty, const char *right )
+slap_dynacl_config(
+       const char *fname,
+       int lineno,
+       Access *b,
+       const char *name,
+       const char *opts,
+       slap_style_t sty,
+       const char *right )
 {
        slap_dynacl_t   *da, *tmp;
        int             rc = 0;
@@ -92,7 +100,7 @@ slap_dynacl_config( const char *fname, int lineno, Access *b, const char *name,
        *tmp = *da;
 
        if ( tmp->da_parse ) {
-               rc = ( *tmp->da_parse )( fname, lineno, sty, right, &tmp->da_private );
+               rc = ( *tmp->da_parse )( fname, lineno, opts, sty, right, &tmp->da_private );
                if ( rc ) {
                        ch_free( tmp );
                        return rc;
@@ -438,8 +446,9 @@ parse_acl(
                                                acl_usage();
                                        }
 
-                               } else if ( strcasecmp( left, "attr" ) == 0
-                                               || strcasecmp( left, "attrs" ) == 0 ) {
+                               } else if ( strcasecmp( left, "attr" ) == 0             /* TOLERATED */
+                                               || strcasecmp( left, "attrs" ) == 0 )   /* DOCUMENTED */
+                               {
                                        a->acl_attrs = str2anlist( a->acl_attrs,
                                                right, "," );
                                        if ( a->acl_attrs == NULL ) {
@@ -450,6 +459,8 @@ parse_acl(
                                        }
 
                                } else if ( strncasecmp( left, "val", 3 ) == 0 ) {
+                                       char    *mr;
+                                       
                                        if ( !BER_BVISEMPTY( &a->acl_attrval ) ) {
                                                fprintf( stderr,
                                "%s: line %d: attr val already specified in to clause.\n",
@@ -463,62 +474,121 @@ parse_acl(
                                                        fname, lineno );
                                                acl_usage();
                                        }
+
                                        ber_str2bv( right, 0, 1, &a->acl_attrval );
-                                       if ( style && strcasecmp( style, "regex" ) == 0 ) {
-                                               int e = regcomp( &a->acl_attrval_re, a->acl_attrval.bv_val,
-                                                       REG_EXTENDED | REG_ICASE | REG_NOSUB );
-                                               if ( e ) {
-                                                       char buf[512];
-                                                       regerror( e, &a->acl_attrval_re, buf, sizeof(buf) );
+                                       a->acl_attrval_style = ACL_STYLE_BASE;
+
+                                       mr = strchr( left, '/' );
+                                       if ( mr != NULL ) {
+                                               mr[ 0 ] = '\0';
+                                               mr++;
+
+                                               a->acl_attrval_mr = mr_find( mr );
+                                               if ( a->acl_attrval_mr == NULL ) {
                                                        fprintf( stderr, "%s: line %d: "
-                                                               "regular expression \"%s\" bad because of %s\n",
-                                                               fname, lineno, right, buf );
+                                                               "invalid matching rule \"%s\".\n",
+                                                               fname, lineno, mr );
                                                        acl_usage();
                                                }
-                                               a->acl_attrval_style = ACL_STYLE_REGEX;
-                                       } else {
-                                               /* FIXME: if the attribute has DN syntax, we might
-                                                * allow one, subtree and children styles as well */
-                                               if ( !strcasecmp( style, "exact" ) ) {
-                                                       a->acl_attrval_style = ACL_STYLE_BASE;
 
-                                               } else if ( a->acl_attrs[0].an_desc->ad_type->
-                                                       sat_syntax == slap_schema.si_syn_distinguishedName )
+                                               if( !mr_usable_with_at( a->acl_attrval_mr, a->acl_attrs[ 0 ].an_desc->ad_type ) )
                                                {
-                                                       if ( !strcasecmp( style, "baseObject" ) ||
-                                                               !strcasecmp( style, "base" ) )
-                                                       {
+                                                       fprintf( stderr, "%s: line %d: "
+                                                               "matching rule \"%s\" use "
+                                                               "with attr \"%s\" not appropriate.\n",
+                                                               fname, lineno, mr,
+                                                               a->acl_attrs[ 0 ].an_name.bv_val );
+                                                       acl_usage();
+                                               }
+                                       }
+                                       
+                                       if ( style != NULL ) {
+                                               if ( strcasecmp( style, "regex" ) == 0 ) {
+                                                       int e = regcomp( &a->acl_attrval_re, a->acl_attrval.bv_val,
+                                                               REG_EXTENDED | REG_ICASE | REG_NOSUB );
+                                                       if ( e ) {
+                                                               char buf[512];
+                                                               regerror( e, &a->acl_attrval_re, buf, sizeof(buf) );
+                                                               fprintf( stderr, "%s: line %d: "
+                                                                       "regular expression \"%s\" bad because of %s\n",
+                                                                       fname, lineno, right, buf );
+                                                               acl_usage();
+                                                       }
+                                                       a->acl_attrval_style = ACL_STYLE_REGEX;
+
+                                               } else {
+                                                       /* FIXME: if the attribute has DN syntax, we might
+                                                        * allow one, subtree and children styles as well */
+                                                       if ( !strcasecmp( style, "base" ) ||
+                                                               !strcasecmp( style, "exact" ) ) {
                                                                a->acl_attrval_style = ACL_STYLE_BASE;
-                                                       } else if ( !strcasecmp( style, "onelevel" ) ||
-                                                               !strcasecmp( style, "one" ) )
-                                                       {
-                                                               a->acl_attrval_style = ACL_STYLE_ONE;
-                                                       } else if ( !strcasecmp( style, "subtree" ) ||
-                                                               !strcasecmp( style, "sub" ) )
+
+                                                       } else if ( a->acl_attrs[0].an_desc->ad_type->
+                                                               sat_syntax == slap_schema.si_syn_distinguishedName )
                                                        {
-                                                               a->acl_attrval_style = ACL_STYLE_SUBTREE;
-                                                       } else if ( !strcasecmp( style, "children" ) ) {
-                                                               a->acl_attrval_style = ACL_STYLE_CHILDREN;
+                                                               struct berval   bv;
+
+                                                               if ( !strcasecmp( style, "baseObject" ) ||
+                                                                       !strcasecmp( style, "base" ) )
+                                                               {
+                                                                       a->acl_attrval_style = ACL_STYLE_BASE;
+                                                               } else if ( !strcasecmp( style, "onelevel" ) ||
+                                                                       !strcasecmp( style, "one" ) )
+                                                               {
+                                                                       a->acl_attrval_style = ACL_STYLE_ONE;
+                                                               } else if ( !strcasecmp( style, "subtree" ) ||
+                                                                       !strcasecmp( style, "sub" ) )
+                                                               {
+                                                                       a->acl_attrval_style = ACL_STYLE_SUBTREE;
+                                                               } else if ( !strcasecmp( style, "children" ) ) {
+                                                                       a->acl_attrval_style = ACL_STYLE_CHILDREN;
+                                                               } else {
+                                                                       fprintf( stderr, 
+                                                                               "%s: line %d: unknown val.<style> \"%s\" "
+                                                                               "for attributeType \"%s\" with DN syntax; "
+                                                                               "using \"base\"\n",
+                                                                               fname, lineno, style,
+                                                                               a->acl_attrs[0].an_desc->ad_cname.bv_val );
+                                                                       a->acl_attrval_style = ACL_STYLE_BASE;
+                                                               }
+
+                                                               bv = a->acl_attrval;
+                                                               rc = dnNormalize( 0, NULL, NULL, &bv, &a->acl_attrval, NULL );
+                                                               if ( rc != LDAP_SUCCESS ) {
+                                                                       fprintf( stderr, 
+                                                                               "%s: line %d: unable to normalize DN \"%s\" "
+                                                                               "for attributeType \"%s\" (%d).\n",
+                                                                               fname, lineno, bv.bv_val,
+                                                                               a->acl_attrs[0].an_desc->ad_cname.bv_val, rc );
+                                                                       acl_usage();
+                                                               }
+                                                               ber_memfree( bv.bv_val );
+
                                                        } else {
                                                                fprintf( stderr, 
                                                                        "%s: line %d: unknown val.<style> \"%s\" "
-                                                                       "for attributeType \"%s\" with DN syntax; "
-                                                                       "using \"base\"\n",
+                                                                       "for attributeType \"%s\"; using \"exact\"\n",
                                                                        fname, lineno, style,
                                                                        a->acl_attrs[0].an_desc->ad_cname.bv_val );
                                                                a->acl_attrval_style = ACL_STYLE_BASE;
                                                        }
+                                               }
+                                       }
 
-                                               } else {
-                                                       fprintf( stderr, 
-                                                               "%s: line %d: unknown val.<style> \"%s\" "
-                                                               "for attributeType \"%s\"; using \"exact\"\n",
-                                                               fname, lineno, style,
-                                                               a->acl_attrs[0].an_desc->ad_cname.bv_val );
-                                                       a->acl_attrval_style = ACL_STYLE_BASE;
+                                       /* Check for appropriate matching rule */
+                                       if ( a->acl_attrval_style != ACL_STYLE_REGEX ) {
+                                               if ( a->acl_attrval_mr == NULL ) {
+                                                       a->acl_attrval_mr = a->acl_attrs[ 0 ].an_desc->ad_type->sat_equality;
+                                               }
+
+                                               if ( a->acl_attrval_mr == NULL ) {
+                                                       fprintf( stderr, "%s: line %d: "
+                                                               "attr \"%s\" must have an EQUALITY matching rule.\n",
+                                                               fname, lineno, a->acl_attrs[ 0 ].an_name.bv_val );
+                                                       acl_usage();
                                                }
                                        }
-                                       
+
                                } else {
                                        fprintf( stderr,
                                                "%s: line %d: expecting <what> got \"%s\"\n",
@@ -532,6 +602,7 @@ parse_acl(
                        {
                                free( a->acl_dn_pat.bv_val );
                                BER_BVZERO( &a->acl_dn_pat );
+                               a->acl_dn_style = ACL_STYLE_REGEX;
                        }
                        
                        if ( !BER_BVISEMPTY( &a->acl_dn_pat ) ||
@@ -690,8 +761,12 @@ parse_acl(
                                        case ACL_STYLE_REGEX:
                                                fprintf( stderr, "%s: line %d: "
                                                        "\"regex\" style implies "
-                                                       "\"expand\" modifier (ignored)\n",
+                                                       "\"expand\" modifier" 
+                                                       SLAPD_CONF_UNKNOWN_IGNORED ".\n",
                                                        fname, lineno );
+#ifdef SLAPD_CONF_UNKNOWN_BAILOUT
+                                               acl_usage();
+#endif /* SLAPD_CONF_UNKNOWN_BAILOUT */
                                                break;
 
                                        case ACL_STYLE_EXPAND:
@@ -700,8 +775,12 @@ parse_acl(
                                                fprintf( stderr, "%s: line %d: "
                                                        "\"expand\" style used "
                                                        "in conjunction with "
-                                                       "\"expand\" modifier (ignored)\n",
+                                                       "\"expand\" modifier"
+                                                       SLAPD_CONF_UNKNOWN_IGNORED ".\n",
                                                        fname, lineno );
+#ifdef SLAPD_CONF_UNKNOWN_BAILOUT
+                                               acl_usage();
+#endif /* SLAPD_CONF_UNKNOWN_BAILOUT */
 #endif
                                                break;
 
@@ -847,7 +926,34 @@ parse_acl(
                                                bdn->a_pat = bv;
                                        }
                                        bdn->a_style = sty;
-                                       bdn->a_expand = expand;
+                                       if ( expand ) {
+                                               char    *exp;
+                                               int     gotit = 0;
+
+                                               for ( exp = strchr( bdn->a_pat.bv_val, '$' );
+                                                               exp && exp - bdn->a_pat.bv_val < bdn->a_pat.bv_len;
+                                                               exp = strchr( exp, '$' ) )
+                                               {
+                                                       if ( isdigit( exp[ 1 ] ) ) {
+                                                               gotit = 1;
+                                                               break;
+                                                       }
+                                               }
+
+                                               if ( gotit == 1 ) {
+                                                       bdn->a_expand = expand;
+
+                                               } else {
+                                                       fprintf( stderr,
+                                                               "%s: line %d: \"expand\" used "
+                                                               "with no expansions in \"pattern\""
+                                                               SLAPD_CONF_UNKNOWN_IGNORED ".\n",
+                                                               fname, lineno );
+#ifdef SLAPD_CONF_UNKNOWN_BAILOUT
+                                                       acl_usage();
+#endif /* SLAPD_CONF_UNKNOWN_BAILOUT */
+                                               } 
+                                       }
                                        if ( sty == ACL_STYLE_SELF ) {
                                                bdn->a_self_level = level;
 
@@ -862,12 +968,12 @@ parse_acl(
                                                        fprintf( stderr,
                                                                "%s: line %d: \"onelevel\" should be used "
                                                                "instead of \"level{1}\" in by DN clause\n",
-                                                               fname, lineno, 0 );
+                                                               fname, lineno );
                                                } else if ( level == 0 && sty == ACL_STYLE_LEVEL ) {
                                                        fprintf( stderr,
                                                                "%s: line %d: \"base\" should be used "
                                                                "instead of \"level{0}\" in by DN clause\n",
-                                                               fname, lineno, 0 );
+                                                               fname, lineno );
                                                }
 
                                                bdn->a_level = level;
@@ -935,7 +1041,7 @@ parse_acl(
                                                fprintf( stderr, "%s: line %d: "
                                                        "deprecated group style \"regex\"; "
                                                        "use \"expand\" instead\n",
-                                                       fname, lineno, style );
+                                                       fname, lineno );
                                                sty = ACL_STYLE_EXPAND;
                                                break;
 
@@ -1010,7 +1116,7 @@ parse_acl(
                                                }
 
                                        } else {
-                                               b->a_group_oc = oc_find(SLAPD_GROUP_CLASS);
+                                               b->a_group_oc = oc_find( SLAPD_GROUP_CLASS );
 
                                                if( b->a_group_oc == NULL ) {
                                                        fprintf( stderr,
@@ -1391,17 +1497,23 @@ parse_acl(
 
 #ifdef SLAP_DYNACL
                                {
-                                       char            *name = NULL;
+                                       char            *name = NULL,
+                                                       *opts = NULL;
                                        
                                        if ( strcasecmp( left, "aci" ) == 0 ) {
                                                name = "aci";
                                                
                                        } else if ( strncasecmp( left, "dynacl/", STRLENOF( "dynacl/" ) ) == 0 ) {
                                                name = &left[ STRLENOF( "dynacl/" ) ];
+                                               opts = strchr( name, '/' );
+                                               if ( opts ) {
+                                                       opts[ 0 ] = '\0';
+                                                       opts++;
+                                               }
                                        }
 
                                        if ( name ) {
-                                               if ( slap_dynacl_config( fname, lineno, b, name, sty, right ) ) {
+                                               if ( slap_dynacl_config( fname, lineno, b, name, opts, sty, right ) ) {
                                                        fprintf( stderr, "%s: line %d: "
                                                                "unable to configure dynacl \"%s\"\n",
                                                                fname, lineno, name );
@@ -1440,7 +1552,7 @@ parse_acl(
                                                }
 
                                        } else {
-                                               b->a_aci_at = slap_schema.si_ad_aci;
+                                               b->a_aci_at = slap_ad_aci;
                                        }
 
                                        if( !is_at_syntax( b->a_aci_at->ad_type,
@@ -2010,7 +2122,7 @@ acl_usage( void )
                "<access clause> ::= access to <what> "
                                "[ by <who> <access> [ <control> ] ]+ \n"
                "<what> ::= * | [dn[.<dnstyle>]=<DN>] [filter=<filter>] [attrs=<attrlist>]\n"
-               "<attrlist> ::= <attr> [val[.<attrstyle>]=<value>] | <attr> , <attrlist>\n"
+               "<attrlist> ::= <attr> [val[/matchingRule][.<attrstyle>]=<value>] | <attr> , <attrlist>\n"
                "<attr> ::= <attrname> | entry | children\n",
                "<who> ::= [ * | anonymous | users | self | dn[.<dnstyle>]=<DN> ]\n"
                        "\t[ realanonymous | realusers | realself | realdn[.<dnstyle>]=<DN> ]\n"
@@ -2019,12 +2131,13 @@ acl_usage( void )
                        "\t[group[/<objectclass>[/<attrname>]][.<style>]=<group>]\n"
                        "\t[peername[.<peernamestyle>]=<peer>] [sockname[.<style>]=<name>]\n"
                        "\t[domain[.<domainstyle>]=<domain>] [sockurl[.<style>]=<url>]\n"
-#ifdef SLAPD_ACI_ENABLED
-                       "\t[aci=[<attrname>]]\n"
-#endif
 #ifdef SLAP_DYNACL
-                       "\t[dynacl/<name>[.<dynstyle>][=<pattern>]]\n"
-#endif /* SLAP_DYNACL */
+                       "\t[dynacl/<name>[/<options>][.<dynstyle>][=<pattern>]]\n"
+#else /* ! SLAP_DYNACL */
+#ifdef SLAPD_ACI_ENABLED
+                       "\t[aci[=<attrname>]]\n"
+#endif /* SLAPD_ACI_ENABLED */
+#endif /* ! SLAP_DYNACL */
                        "\t[ssf=<n>] [transport_ssf=<n>] [tls_ssf=<n>] [sasl_ssf=<n>]\n",
                "<style> ::= exact | regex | base(Object)\n"
                "<dnstyle> ::= base(Object) | one(level) | sub(tree) | children | "
@@ -2037,6 +2150,12 @@ acl_usage( void )
                "<level> ::= none|disclose|auth|compare|search|read|{write|add|delete}|manage\n"
                "<priv> ::= {=|+|-}{0|d|x|c|s|r|{w|a|z}|m}+\n"
                "<control> ::= [ stop | continue | break ]\n"
+#ifdef SLAP_DYNACL
+#ifdef SLAPD_ACI_ENABLED
+               "dynacl:\n"
+               "\t<name>=ACI\t<pattern>=<attrname>\n"
+#endif /* SLAPD_ACI_ENABLED */
+#endif /* ! SLAP_DYNACL */
        );
        exit( EXIT_FAILURE );
 }
@@ -2151,6 +2270,22 @@ access_free( Access *a )
        if ( !BER_BVISNULL( &a->a_group_pat ) ) {
                free( a->a_group_pat.bv_val );
        }
+#ifdef SLAP_DYNACL
+       if ( a->a_dynacl != NULL ) {
+               slap_dynacl_t   *da;
+               for ( da = a->a_dynacl; da; ) {
+                       slap_dynacl_t   *tmp = da;
+
+                       da = da->da_next;
+
+                       if ( tmp->da_destroy ) {
+                               tmp->da_destroy( tmp->da_private );
+                       }
+
+                       ch_free( tmp );
+               }
+       }
+#endif /* SLAP_DYNACL */
        free( a );
 }
 
@@ -2164,6 +2299,9 @@ acl_free( AccessControl *a )
                filter_free( a->acl_filter );
        }
        if ( !BER_BVISNULL( &a->acl_dn_pat ) ) {
+               if ( a->acl_dn_style == ACL_STYLE_REGEX ) {
+                       regfree( &a->acl_dn_re );
+               }
                free ( a->acl_dn_pat.bv_val );
        }
        if ( a->acl_attrs ) {
@@ -2191,7 +2329,7 @@ acl_destroy( AccessControl *a, AccessControl *end )
 {
        AccessControl *n;
 
-       for (; a && a!= end; a=n) {
+       for ( ; a && a != end; a = n ) {
                n = a->acl_next;
                acl_free( a );
        }
@@ -2308,7 +2446,10 @@ dnaccess2text( slap_dn_access *bdn, char *ptr, int is_realdn )
 
        } else {
                ptr = lutil_strcopy( ptr, "dn." );
-               ptr = lutil_strcopy( ptr, style_strings[bdn->a_style] );
+               if ( bdn->a_style == ACL_STYLE_BASE )
+                       ptr = lutil_strcopy( ptr, style_base );
+               else 
+                       ptr = lutil_strcopy( ptr, style_strings[bdn->a_style] );
                if ( bdn->a_style == ACL_STYLE_LEVEL ) {
                        int n = sprintf( ptr, "{%d}", bdn->a_level );
                        if ( n > 0 ) {
@@ -2323,16 +2464,6 @@ dnaccess2text( slap_dn_access *bdn, char *ptr, int is_realdn )
                ptr = lutil_strcopy( ptr, bdn->a_pat.bv_val );
                *ptr++ = '"';
        }
-
-       if ( bdn->a_at != NULL ) {
-               *ptr++ = ' ';
-               if ( is_realdn ) {
-                       ptr = lutil_strcopy( ptr, "real" );
-               }
-               ptr = lutil_strcopy( ptr, "dnattr=" );
-               ptr = lutil_strcopy( ptr, bdn->a_at->ad_cname.bv_val );
-       }
-
        return ptr;
 }
 
@@ -2346,18 +2477,26 @@ access2text( Access *b, char *ptr )
        if ( !BER_BVISEMPTY( &b->a_dn_pat ) ) {
                ptr = dnaccess2text( &b->a_dn, ptr, 0 );
        }
+       if ( b->a_dn_at ) {
+               ptr = lutil_strcopy( ptr, " dnattr=" );
+               ptr = lutil_strcopy( ptr, b->a_dn_at->ad_cname.bv_val );
+       }
 
        if ( !BER_BVISEMPTY( &b->a_realdn_pat ) ) {
                ptr = dnaccess2text( &b->a_realdn, ptr, 1 );
        }
+       if ( b->a_realdn_at ) {
+               ptr = lutil_strcopy( ptr, " realdnattr=" );
+               ptr = lutil_strcopy( ptr, b->a_realdn_at->ad_cname.bv_val );
+       }
 
        if ( !BER_BVISEMPTY( &b->a_group_pat ) ) {
                ptr = lutil_strcopy( ptr, " group/" );
                ptr = lutil_strcopy( ptr, b->a_group_oc ?
-                       b->a_group_oc->soc_cname.bv_val : "groupOfNames" );
+                       b->a_group_oc->soc_cname.bv_val : SLAPD_GROUP_CLASS );
                *ptr++ = '/';
                ptr = lutil_strcopy( ptr, b->a_group_at ?
-                       b->a_group_at->ad_cname.bv_val : "member" );
+                       b->a_group_at->ad_cname.bv_val : SLAPD_GROUP_ATTR );
                *ptr++ = '.';
                ptr = lutil_strcopy( ptr, style_strings[b->a_group_style] );
                *ptr++ = '=';
@@ -2367,30 +2506,52 @@ access2text( Access *b, char *ptr )
        }
 
        if ( !BER_BVISEMPTY( &b->a_peername_pat ) ) {
-               ptr = lutil_strcopy( ptr, " peername=\"" );
+               ptr = lutil_strcopy( ptr, " peername" );
+               *ptr++ = '.';
+               ptr = lutil_strcopy( ptr, style_strings[b->a_peername_style] );
+               *ptr++ = '=';
+               *ptr++ = '"';
                ptr = lutil_strcopy( ptr, b->a_peername_pat.bv_val );
                *ptr++ = '"';
        }
 
        if ( !BER_BVISEMPTY( &b->a_sockname_pat ) ) {
-               ptr = lutil_strcopy( ptr, " sockname=\"" );
+               ptr = lutil_strcopy( ptr, " sockname" );
+               *ptr++ = '.';
+               ptr = lutil_strcopy( ptr, style_strings[b->a_sockname_style] );
+               *ptr++ = '=';
+               *ptr++ = '"';
                ptr = lutil_strcopy( ptr, b->a_sockname_pat.bv_val );
                *ptr++ = '"';
        }
 
        if ( !BER_BVISEMPTY( &b->a_domain_pat ) ) {
-               ptr = lutil_strcopy( ptr, " domain=" );
+               ptr = lutil_strcopy( ptr, " domain" );
+               *ptr++ = '.';
+               ptr = lutil_strcopy( ptr, style_strings[b->a_domain_style] );
+               if ( b->a_domain_expand ) {
+                       ptr = lutil_strcopy( ptr, ",expand" );
+               }
+               *ptr++ = '=';
                ptr = lutil_strcopy( ptr, b->a_domain_pat.bv_val );
        }
 
        if ( !BER_BVISEMPTY( &b->a_sockurl_pat ) ) {
-               ptr = lutil_strcopy( ptr, " sockurl=\"" );
+               ptr = lutil_strcopy( ptr, " sockurl" );
+               *ptr++ = '.';
+               ptr = lutil_strcopy( ptr, style_strings[b->a_sockurl_style] );
+               *ptr++ = '=';
+               *ptr++ = '"';
                ptr = lutil_strcopy( ptr, b->a_sockurl_pat.bv_val );
                *ptr++ = '"';
        }
 
        if ( !BER_BVISEMPTY( &b->a_set_pat ) ) {
-               ptr = lutil_strcopy( ptr, " set=\"" );
+               ptr = lutil_strcopy( ptr, " set" );
+               *ptr++ = '.';
+               ptr = lutil_strcopy( ptr, style_strings[b->a_set_style] );
+               *ptr++ = '=';
+               *ptr++ = '"';
                ptr = lutil_strcopy( ptr, b->a_set_pat.bv_val );
                *ptr++ = '"';
        }
@@ -2401,8 +2562,9 @@ access2text( Access *b, char *ptr )
 
                for ( da = b->a_dynacl; da; da = da->da_next ) {
                        if ( da->da_unparse ) {
-                               struct berval bv;
+                               struct berval bv = BER_BVNULL;
                                (void)( *da->da_unparse )( da->da_private, &bv );
+                               assert( !BER_BVISNULL( &bv ) );
                                ptr = lutil_strcopy( ptr, bv.bv_val );
                                ch_free( bv.bv_val );
                        }
@@ -2476,7 +2638,10 @@ acl_unparse( AccessControl *a, struct berval *bv )
        if ( !BER_BVISNULL( &a->acl_dn_pat ) ) {
                to++;
                ptr = lutil_strcopy( ptr, " dn." );
-               ptr = lutil_strcopy( ptr, style_strings[a->acl_dn_style] );
+               if ( a->acl_dn_style == ACL_STYLE_BASE )
+                       ptr = lutil_strcopy( ptr, style_base );
+               else
+                       ptr = lutil_strcopy( ptr, style_strings[a->acl_dn_style] );
                *ptr++ = '=';
                *ptr++ = '"';
                ptr = lutil_strcopy( ptr, a->acl_dn_pat.bv_val );
@@ -2518,7 +2683,12 @@ acl_unparse( AccessControl *a, struct berval *bv )
        if ( !BER_BVISEMPTY( &a->acl_attrval ) ) {
                to++;
                ptr = lutil_strcopy( ptr, " val." );
-               ptr = lutil_strcopy( ptr, style_strings[a->acl_attrval_style] );
+               if ( a->acl_attrval_style == ACL_STYLE_BASE &&
+                       a->acl_attrs[0].an_desc->ad_type->sat_syntax ==
+                               slap_schema.si_syn_distinguishedName )
+                       ptr = lutil_strcopy( ptr, style_base );
+               else
+                       ptr = lutil_strcopy( ptr, style_strings[a->acl_attrval_style] );
                *ptr++ = '=';
                *ptr++ = '"';
                ptr = lutil_strcopy( ptr, a->acl_attrval.bv_val );