]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-meta/config.c
Improve schema error messages
[openldap] / servers / slapd / back-meta / config.c
index 72f557d08f972143884c69468d0fba64e55f44d6..107ac5cadb9cc1c350fd99aa8f8527c8ec90e573 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 1998-2001 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2002 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  *
  * Copyright 2001, Pierangelo Masarati, All rights reserved. <ando@sys-net.it>
 #include "../back-ldap/back-ldap.h"
 #include "back-meta.h"
 
-extern int
-suffix_massage_config(
-               struct rewrite_info *info,
-               int argc,
-               char **argv
-);
-
 static struct metatarget *
 new_target( void )
 {
@@ -99,20 +92,7 @@ new_target( void )
                 return NULL;
        }
 
-       mapping = ch_calloc( 2, sizeof( struct ldapmapping ) );
-        if ( mapping == NULL ) {
-               free( lt );
-               return NULL;
-       }
-       mapping->src = ch_strdup( "objectClass" );
-       mapping->dst = ch_strdup( "objectClass" );
-       mapping[ 1 ].src = mapping->src;
-       mapping[ 1 ].dst = mapping->dst;
-
-       avl_insert( &lt->at_map.map, ( caddr_t )mapping, 
-                       mapping_cmp, mapping_dup );
-       avl_insert( &lt->at_map.remap, ( caddr_t )&mapping[ 1 ],
-                       mapping_cmp, mapping_dup );
+       ldap_back_map_init( &lt->at_map, &mapping );
 
        return lt;
 }
@@ -143,7 +123,7 @@ meta_back_db_config(
 #endif /* uncomment if uri MUST be a branch of suffix */
                LDAPURLDesc     *ludp;
                char            *last;
-               struct berval   dn, *pdn = NULL, *ndn = NULL;
+               struct berval   dn;
                int             rc;
                
                if ( argc != 2 ) {
@@ -248,8 +228,8 @@ meta_back_db_config(
                 * or worked out, at least, in some manner
                 */
                for ( j = 0; j < i-1; j++ ) {
-                       if ( strcmp( li->targets[ i ]->suffix.bv_val,
-                                       li->targets[ j ]->suffix.bv_val ) == 0 ) {
+                       if ( dn_match( &li->targets[ i ]->suffix,
+                                       &li->targets[ j ]->suffix ) ) {
                                fprintf( stderr,
        "%s: line %d: naming context \"%s\" already used"
        " in \"uri <protocol>://<server>[:port]/<naming context>\" line\n",
@@ -261,9 +241,11 @@ meta_back_db_config(
                
                ldap_free_urldesc( ludp );
 
+#if 0
                fprintf(stderr, "%s: line %d: URI \"%s\", suffix \"%s\"\n",
                        fname, lineno, li->targets[ i ]->uri, 
                        li->targets[ i ]->psuffix.bv_val );
+#endif
                
        /* default target directive */
        } else if ( strcasecmp( argv[ 0 ], "default-target" ) == 0 ) {
@@ -319,7 +301,7 @@ meta_back_db_config(
        /* name to use for meta_back_group */
        } else if ( strcasecmp( argv[ 0 ], "binddn" ) == 0 ) {
                int             i = li->ntargets-1;
-               struct berval   dn, *ndn = NULL;
+               struct berval   dn;
 
                if ( i < 0 ) {
                        fprintf( stderr,
@@ -359,12 +341,12 @@ meta_back_db_config(
                            fname, lineno );
                        return 1;
                }
-               li->targets[ i ]->bindpw = ber_bvstrdup( argv[ 1 ] );
+               ber_str2bv( argv[ 1 ], 0L, 1, &li->targets[ i ]->bindpw );
                
        /* name to use as pseudo-root dn */
        } else if ( strcasecmp( argv[ 0 ], "pseudorootdn" ) == 0 ) {
                int             i = li->ntargets-1;
-               struct berval   dn, *ndn = NULL;
+               struct berval   dn;
 
                if ( i < 0 ) {
                        fprintf( stderr,
@@ -404,13 +386,13 @@ meta_back_db_config(
                            fname, lineno );
                        return 1;
                }
-               li->targets[ i ]->pseudorootpw = ber_bvstrdup( argv[ 1 ] );
+               ber_str2bv( argv[ 1 ], 0L, 1, &li->targets[ i ]->pseudorootpw );
        
        /* dn massaging */
        } else if ( strcasecmp( argv[ 0 ], "suffixmassage" ) == 0 ) {
                BackendDB       *tmp_be;
                int             i = li->ntargets-1;
-               struct berval   dn, ndn;
+               struct berval   dn, nvnc, pvnc, nrnc, prnc;
 
                if ( i < 0 ) {
                        fprintf( stderr,
@@ -439,41 +421,49 @@ meta_back_db_config(
 
                dn.bv_val = argv[ 1 ];
                dn.bv_len = strlen( argv[ 1 ] );
-               if ( dnNormalize2( NULL, &dn, &ndn ) != LDAP_SUCCESS ) {
+               if ( dnPrettyNormal( NULL, &dn, &pvnc, &nvnc ) != LDAP_SUCCESS ) {
                        fprintf( stderr, "%s: line %d: "
                                        "suffix '%s' is invalid\n",
                                        fname, lineno, argv[ 1 ] );
                        return 1;
                }
                
-               tmp_be = select_backend( &ndn, 0, 0 );
-               free( ndn.bv_val );
+               tmp_be = select_backend( &nvnc, 0, 0 );
                if ( tmp_be != NULL && tmp_be != be ) {
                        fprintf( stderr, 
        "%s: line %d: suffix already in use by another backend in"
        " \"suffixMassage <suffix> <massaged suffix>\"\n",
                                fname, lineno );
+                       free( pvnc.bv_val );
+                       free( nvnc.bv_val );
                        return 1;                                               
                }
 
                dn.bv_val = argv[ 2 ];
                dn.bv_len = strlen( argv[ 2 ] );
-               if ( dnNormalize2( NULL, &dn, &ndn ) != LDAP_SUCCESS ) {
+               if ( dnPrettyNormal( NULL, &dn, &prnc, &nrnc ) != LDAP_SUCCESS ) {
                        fprintf( stderr, "%s: line %d: "
                                        "massaged suffix '%s' is invalid\n",
                                        fname, lineno, argv[ 2 ] );
+                       free( pvnc.bv_val );
+                       free( nvnc.bv_val );
                        return 1;
                }
-               
-               tmp_be = select_backend( &ndn, 0, 0 );
-               free( ndn.bv_val );
+       
+#if 0  
+               tmp_be = select_backend( &nrnc, 0, 0 );
                if ( tmp_be != NULL ) {
                        fprintf( stderr,
        "%s: line %d: massaged suffix already in use by another backend in" 
        " \"suffixMassage <suffix> <massaged suffix>\"\n",
                                 fname, lineno );
+                       free( pvnc.bv_val );
+                       free( nvnc.bv_val );
+                       free( prnc.bv_val );
+                       free( nrnc.bv_val );
                         return 1;
                }
+#endif
                
                /*
                 * The suffix massaging is emulated by means of the
@@ -482,7 +472,7 @@ meta_back_db_config(
                 * to the database
                 */
                return suffix_massage_config( li->targets[ i ]->rwinfo,
-                               argc, argv );
+                               &pvnc, &nvnc, &prnc, &nrnc );
                
        /* rewrite stuff ... */
        } else if ( strncasecmp( argv[ 0 ], "rewrite", 7 ) == 0 ) {
@@ -513,7 +503,7 @@ meta_back_db_config(
 
                if ( argc < 3 || argc > 4 ) {
                        fprintf( stderr,
-       "%s: line %d: syntax is \"map {objectclass | attribute} {<source> | *} [<dest> | *]\"\n",
+       "%s: line %d: syntax is \"map {objectclass | attribute} [<local> | *] {<foreign> | *}\"\n",
                                fname, lineno );
                        return 1;
                }
@@ -524,32 +514,23 @@ meta_back_db_config(
                        map = &li->targets[ i ]->at_map;
                } else {
                        fprintf( stderr,
-       "%s: line %d: syntax is \"map {objectclass | attribute} {<source> | *} [<dest> | *]\"\n",
+       "%s: line %d: syntax is \"map {objectclass | attribute} [<local> | *] {<foreign> | *}\"\n",
                                fname, lineno );
                        return 1;
                }
 
-               if ( strcasecmp( argv[ 2 ], "*" ) != 0 ) {
-                       src = argv[ 2 ];
-                       if ( argc < 4 ) {
-                               dst = "";
-                       } else if ( strcasecmp( argv[ 3 ], "*" ) == 0 ) {
-                               dst = src;
-                       } else {
-                               dst = argv[ 3 ];
-                       }
-               } else {
-                       if ( argc < 4 ) {
-                               map->drop_missing = 1;
+               if ( strcmp( argv[ 2 ], "*" ) == 0 ) {
+                       if ( argc < 4 || strcmp( argv[ 3 ], "*" ) == 0 ) {
+                               map->drop_missing = ( argc < 4 );
                                return 0;
                        }
-                       if ( strcasecmp( argv[ 3 ], "*" ) == 0 ) {
-                               map->drop_missing = 0;
-                               return 0;
-                       }
-
-                       src = argv[ 3 ];
-                       dst = src;
+                       src = dst = argv[ 3 ];
+               } else if ( argc < 4 ) {
+                       src = "";
+                       dst = argv[ 2 ];
+               } else {
+                       src = argv[ 2 ];
+                       dst = ( strcmp( argv[ 3 ], "*" ) == 0 ? src : argv[ 3 ] );
                }
 
                if ( ( map == &li->targets[ i ]->at_map )
@@ -567,18 +548,14 @@ meta_back_db_config(
                                fname, lineno );
                        return 1;
                }
-               mapping->src = ch_strdup( src );
-               mapping->dst = ch_strdup( dst );
-               if ( *dst != 0 ) {
-                       mapping[ 1 ].src = mapping->dst;
-                       mapping[ 1 ].dst = mapping->src;
-               } else {
-                       mapping[ 1 ].src = mapping->src;
-                       mapping[ 1 ].dst = mapping->dst;
-               }
+               ber_str2bv( src, 0, 1, &mapping->src );
+               ber_str2bv( dst, 0, 1, &mapping->dst );
+               mapping[ 1 ].src = mapping->dst;
+               mapping[ 1 ].dst = mapping->src;
 
-               if ( avl_find( map->map, ( caddr_t )mapping,
-                               mapping_cmp ) != NULL
+               if ( (*src != '\0' &&
+                         avl_find( map->map, ( caddr_t )mapping,
+                               mapping_cmp ) != NULL)
                        || avl_find( map->remap, ( caddr_t )&mapping[ 1 ],
                                mapping_cmp ) != NULL) {
                        fprintf( stderr,
@@ -587,8 +564,9 @@ meta_back_db_config(
                        return 0;
                }
 
-               avl_insert( &map->map, ( caddr_t )mapping,
-                                       mapping_cmp, mapping_dup );
+               if ( *src != '\0' )
+                       avl_insert( &map->map, ( caddr_t )mapping,
+                                               mapping_cmp, mapping_dup );
                avl_insert( &map->remap, ( caddr_t )&mapping[ 1 ],
                                        mapping_cmp, mapping_dup );