Add NAME check

diff --git a/servers/slapd/at.c b/servers/slapd/at.c
index fd62fe0770cb2aeb9c2608b259263302585519fd..4859040aea85865a111f6ba51b1d72a6e8eb0342 100644 (file)
--- a/servers/slapd/at.c
+++ b/servers/slapd/at.c
@@ -239,11 +239,19 @@ at_add(
        char                    *cname;
 
        if ( at->at_names && at->at_names[0] ) {
+               int i;
+
+               for( i=0; at->at_names[i]; i++ ) {
+                       if( !slap_valid_descr( at->at_names[i] ) ) {
+                               return SLAP_SCHERR_BAD_DESCR;
+                       }
+               }
+
                cname = at->at_names[0];
+
        } else if ( at->at_oid ) {
                cname = at->at_oid;
        } else {
-               cname = "";
                return SLAP_SCHERR_ATTR_INCOMPLETE;
        }
 

diff --git a/servers/slapd/oc.c b/servers/slapd/oc.c
index c27f518029fe214d8e2bf42d9885d48458003035..6433916327a513880f79a9a158b16f3457cc7e2e 100644 (file)
--- a/servers/slapd/oc.c
+++ b/servers/slapd/oc.c
@@ -350,6 +350,16 @@ oc_add(
        ObjectClass     *soc;
        int             code;
 
+       if ( oc->oc_names != NULL ) {
+               int i;
+
+               for( i=0; oc->oc_names[i]; i++ ) {
+                       if( !slap_valid_descr( oc->oc_names[i] ) ) {
+                               return SLAP_SCHERR_BAD_DESCR;
+                       }
+               }
+       }
+
        soc = (ObjectClass *) ch_calloc( 1, sizeof(ObjectClass) );
        AC_MEMCPY( &soc->soc_oclass, oc, sizeof(LDAPObjectClass) );
 

diff --git a/servers/slapd/proto-slap.h b/servers/slapd/proto-slap.h
index 3688cab747732c335e216197f6425bbb9070122c..ba4ebec32a14c556a58688b82da1b18adcd6e1d0 100644 (file)
--- a/servers/slapd/proto-slap.h
+++ b/servers/slapd/proto-slap.h
@@ -13,6 +13,8 @@ LDAP_BEGIN_DECL
 LDAP_SLAPD_F( int ) schema_init_done;
 LDAP_SLAPD_F( struct slap_internal_schema ) slap_schema;
 
+LDAP_SLAPD_F( int ) slap_valid_descr( const char * );
+
 LDAP_SLAPD_F (int) slap_str2ad LDAP_P((
        const char *,
        AttributeDescription **ad,

diff --git a/servers/slapd/schemaparse.c b/servers/slapd/schemaparse.c
index 5370bf644b1df094cc4f5c81946e10584d9f3629..cd73b41b0a69b70128263a1f11e88766249ae7a9 100644 (file)
--- a/servers/slapd/schemaparse.c
+++ b/servers/slapd/schemaparse.c
@@ -37,7 +37,8 @@ static char *const err2text[] = {
        "MatchingRule not found",
        "Syntax not found",
        "Syntax required",
-       "Qualifier not supported"
+       "Qualifier not supported",
+       "Invalid NAME"
 };
 
 char *
@@ -50,6 +51,24 @@ scherr2str(int code)
        }
 }
 
+/* check schema descr validity */
+int slap_valid_descr( const char *descr )
+{
+       int i=0;
+
+       if( !DESC_LEADCHAR( descr[i] ) ) {
+               return 0;
+       }
+
+       while( descr[++i] ) {
+               if( !DESC_CHAR( descr[i] ) ) {
+                       return 0;
+               }
+       }
+
+       return 1;
+}
+
 
 /* OID Macros */
 

diff --git a/servers/slapd/slap.h b/servers/slapd/slap.h
index bae2b95a3c700b40da97c23ef695265b0ddb0c92..b5d26bb1ad16a60201e885e03cca07f857f05e10 100644 (file)
--- a/servers/slapd/slap.h
+++ b/servers/slapd/slap.h
@@ -217,6 +217,7 @@ typedef struct slap_ssf_set {
 #define SLAP_SCHERR_SYN_NOT_FOUND      13
 #define SLAP_SCHERR_MR_INCOMPLETE      14
 #define SLAP_SCHERR_NOT_SUPPORTED      15
+#define SLAP_SCHERR_BAD_DESCR  16
 
 typedef struct slap_oid_macro {
        struct berval som_oid;