]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-sql/schema-map.c
cleanup
[openldap] / servers / slapd / back-sql / schema-map.c
index 4f68da7f7b611724ed75bba16c123f48919c9628..3f75bb84ad5f4a8fccd3ed26822a4a80ada6a80a 100644 (file)
@@ -1,7 +1,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1999-2003 The OpenLDAP Foundation.
+ * Copyright 1999-2004 The OpenLDAP Foundation.
  * Portions Copyright 1999 Dmitry Kovalev.
  * All rights reserved.
  *
@@ -33,6 +33,8 @@
 #include "schema-map.h"
 #include "util.h"
 
+#define BACKSQL_DUPLICATE      (-1)
+
 /*
  * Uses the pointer to the ObjectClass structure
  */
@@ -40,14 +42,16 @@ static int
 backsql_cmp_oc( const void *v_m1, const void *v_m2 )
 {
        const backsql_oc_map_rec *m1 = v_m1, *m2 = v_m2;
-       return SLAP_PTRCMP( m1->oc, m2->oc );
+
+       return SLAP_PTRCMP( m1->bom_oc, m2->bom_oc );
 }
 
 static int
 backsql_cmp_oc_id( const void *v_m1, const void *v_m2 )
 {
        const backsql_oc_map_rec *m1 = v_m1, *m2 = v_m2;
-       return ( m1->id < m2->id ? -1 : ( m1->id > m2->id ? 1 : 0 ) );
+
+       return ( m1->bom_id < m2->bom_id ? -1 : ( m1->bom_id > m2->bom_id ? 1 : 0 ) );
 }
 
 /*
@@ -57,7 +61,22 @@ static int
 backsql_cmp_attr( const void *v_m1, const void *v_m2 )
 {
        const backsql_at_map_rec *m1 = v_m1, *m2 = v_m2;
-       return SLAP_PTRCMP( m1->ad, m2->ad );
+
+       return SLAP_PTRCMP( m1->bam_ad, m2->bam_ad );
+}
+
+int
+backsql_dup_attr( void *v_m1, void *v_m2 )
+{
+       backsql_at_map_rec *m1 = v_m1, *m2 = v_m2;
+
+       assert( m1->bam_ad == m2->bam_ad );
+       
+       for ( ; m1->bam_next ; m1 = m1->bam_next );
+       m1->bam_next = m2;
+       m2->bam_next = NULL;
+
+       return BACKSQL_DUPLICATE;
 }
 
 static int
@@ -69,24 +88,24 @@ backsql_make_attr_query(
 
        backsql_strfcat( &bb, "lblblblbcbl", 
                        (ber_len_t)sizeof( "SELECT " ) - 1, "SELECT ", 
-                       &at_map->sel_expr, 
+                       &at_map->bam_sel_expr, 
                        (ber_len_t)sizeof( " AS " ) - 1, " AS ", 
-                       &at_map->ad->ad_cname,
+                       &at_map->bam_ad->ad_cname,
                        (ber_len_t)sizeof( " FROM " ) - 1, " FROM ", 
-                       &at_map->from_tbls, 
+                       &at_map->bam_from_tbls, 
                        (ber_len_t)sizeof( " WHERE " ) - 1, " WHERE ", 
-                       &oc_map->keytbl,
+                       &oc_map->bom_keytbl,
                        '.', 
-                       &oc_map->keycol,
+                       &oc_map->bom_keycol,
                        (ber_len_t)sizeof( "=?" ) - 1, "=?" );
 
-       if ( at_map->join_where.bv_val != NULL ) {
+       if ( at_map->bam_join_where.bv_val != NULL ) {
                backsql_strfcat( &bb, "lb",
                                (ber_len_t)sizeof( " AND ") - 1, " AND ", 
-                               &at_map->join_where );
+                               &at_map->bam_join_where );
        }
 
-       at_map->query = bb.bb_val.bv_val;
+       at_map->bam_query = bb.bb_val.bv_val;
        
        return 0;
 }
@@ -100,20 +119,21 @@ backsql_add_sysmaps( backsql_oc_map_rec *oc_map )
        struct berbuf           bb;
        
 
-       snprintf( s, sizeof( s ), "%ld", oc_map->id );
+       snprintf( s, sizeof( s ), "%ld", oc_map->bom_id );
        slen = strlen( s );
 
        at_map = (backsql_at_map_rec *)ch_calloc(1, 
                        sizeof( backsql_at_map_rec ) );
-       at_map->ad = slap_schema.si_ad_objectClass;
-       ber_str2bv( "ldap_entry_objclasses.oc_name", 0, 1, &at_map->sel_expr );
+       at_map->bam_ad = slap_schema.si_ad_objectClass;
+       ber_str2bv( "ldap_entry_objclasses.oc_name", 0, 1,
+                       &at_map->bam_sel_expr );
        ber_str2bv( "ldap_entry_objclasses,ldap_entries", 0, 1, 
-                       &at_map->from_tbls );
+                       &at_map->bam_from_tbls );
        
-       bb.bb_len = at_map->from_tbls.bv_len + 1;
-       bb.bb_val = at_map->from_tbls;
-       backsql_merge_from_clause( &bb, &oc_map->keytbl );
-       at_map->from_tbls = bb.bb_val;
+       bb.bb_len = at_map->bam_from_tbls.bv_len + 1;
+       bb.bb_val = at_map->bam_from_tbls;
+       backsql_merge_from_clause( &bb, &oc_map->bom_keytbl );
+       at_map->bam_from_tbls = bb.bb_val;
 
        bb.bb_val.bv_val = NULL;
        bb.bb_val.bv_len = 0;
@@ -121,33 +141,40 @@ backsql_add_sysmaps( backsql_oc_map_rec *oc_map )
        backsql_strfcat( &bb, "lbcbll",
                        (ber_len_t)sizeof( "ldap_entries.id=ldap_entry_objclasses.entry_id and ldap_entries.keyval=" ) - 1,
                                "ldap_entries.id=ldap_entry_objclasses.entry_id and ldap_entries.keyval=",
-                       &oc_map->keytbl, 
+                       &oc_map->bom_keytbl, 
                        '.', 
-                       &oc_map->keycol,
+                       &oc_map->bom_keycol,
                        (ber_len_t)sizeof( " and ldap_entries.oc_map_id=" ) - 1, 
                                " and ldap_entries.oc_map_id=", 
                        slen, s );
 
-       at_map->join_where = bb.bb_val;
-       at_map->add_proc = NULL;
-       at_map->delete_proc = NULL;
-       at_map->param_order = 0;
-       at_map->expect_return = 0;
+       at_map->bam_oc = oc_map->bom_oc;
+       at_map->bam_join_where = bb.bb_val;
+       at_map->bam_add_proc = NULL;
+       at_map->bam_delete_proc = NULL;
+       at_map->bam_param_order = 0;
+       at_map->bam_expect_return = 0;
+       at_map->bam_next = NULL;
 
        backsql_make_attr_query( oc_map, at_map );
-       avl_insert( &oc_map->attrs, at_map, backsql_cmp_attr, NULL );
+       if ( avl_insert( &oc_map->bom_attrs, at_map, backsql_cmp_attr, backsql_dup_attr ) == BACKSQL_DUPLICATE ) {
+               Debug( LDAP_DEBUG_TRACE, "backsql_add_sysmaps(): "
+                               "duplicate attribute \"%s\" in objectClass \"%s\" map\n",
+                               at_map->bam_ad->ad_cname.bv_val,
+                               oc_map->bom_oc->soc_cname.bv_val, 0 );
+       }
 
        at_map = (backsql_at_map_rec *)ch_calloc( 1, 
                        sizeof( backsql_at_map_rec ) );
-       at_map->ad = slap_schema.si_ad_ref;
-       ber_str2bv( "ldap_referrals.url", 0, 1, &at_map->sel_expr );
-       ber_str2bv( "ldap_referrals,ldap_entries", 0, 1, &at_map->from_tbls );
+       at_map->bam_ad = slap_schema.si_ad_ref;
+       ber_str2bv( "ldap_referrals.url", 0, 1, &at_map->bam_sel_expr );
+       ber_str2bv( "ldap_referrals,ldap_entries", 0, 1, &at_map->bam_from_tbls );
        
        bb.bb_val.bv_val = NULL;
        bb.bb_val.bv_len = 0;
-       bb.bb_len = at_map->from_tbls.bv_len + 1;
-       backsql_merge_from_clause( &bb, &oc_map->keytbl );
-       at_map->from_tbls = bb.bb_val;
+       bb.bb_len = at_map->bam_from_tbls.bv_len + 1;
+       backsql_merge_from_clause( &bb, &oc_map->bom_keytbl );
+       at_map->bam_from_tbls = bb.bb_val;
 
        bb.bb_val.bv_val = NULL;
        bb.bb_val.bv_len = 0;
@@ -155,21 +182,28 @@ backsql_add_sysmaps( backsql_oc_map_rec *oc_map )
        backsql_strfcat( &bb, "lbcbll",
                        (ber_len_t)sizeof( "ldap_entries.id=ldap_referrals.entry_id and ldap_entries.keyval=" ) - 1,
                                "ldap_entries.id=ldap_referrals.entry_id and ldap_entries.keyval=",
-                       &oc_map->keytbl, 
+                       &oc_map->bom_keytbl, 
                        '.', 
-                       &oc_map->keycol,
+                       &oc_map->bom_keycol,
                        (ber_len_t)sizeof( " and ldap_entries.oc_map_id=" ) - 1, 
                                " and ldap_entries.oc_map_id=", 
                        slen, s );
 
-       at_map->join_where = bb.bb_val;
-       at_map->add_proc = NULL;
-       at_map->delete_proc = NULL;
-       at_map->param_order = 0;
-       at_map->expect_return = 0;
+       at_map->bam_oc = NULL;
+       at_map->bam_join_where = bb.bb_val;
+       at_map->bam_add_proc = NULL;
+       at_map->bam_delete_proc = NULL;
+       at_map->bam_param_order = 0;
+       at_map->bam_expect_return = 0;
+       at_map->bam_next = NULL;
 
        backsql_make_attr_query( oc_map, at_map );
-       avl_insert( &oc_map->attrs, at_map, backsql_cmp_attr, NULL );
+       if ( avl_insert( &oc_map->bom_attrs, at_map, backsql_cmp_attr, backsql_dup_attr ) == BACKSQL_DUPLICATE ) {
+               Debug( LDAP_DEBUG_TRACE, "backsql_add_sysmaps(): "
+                               "duplicate attribute \"%s\" in objectClass \"%s\" map\n",
+                               at_map->bam_ad->ad_cname.bv_val,
+                               oc_map->bom_oc->soc_cname.bv_val, 0 );
+       }
 
        return 1;
 }
@@ -250,30 +284,30 @@ backsql_load_schema_map( backsql_info *si, SQLHDBC dbh )
                oc_map = (backsql_oc_map_rec *)ch_calloc( 1,
                                sizeof( backsql_oc_map_rec ) );
 
-               oc_map->id = strtol( oc_row.cols[ 0 ], NULL, 0 );
+               oc_map->bom_id = strtol( oc_row.cols[ 0 ], NULL, 0 );
 
-               oc_map->oc = oc_find( oc_row.cols[ 1 ] );
-               if ( oc_map->oc == NULL ) {
+               oc_map->bom_oc = oc_find( oc_row.cols[ 1 ] );
+               if ( oc_map->bom_oc == NULL ) {
                        Debug( LDAP_DEBUG_TRACE, "load_schema_map(): "
                                "objectClass '%s' is not defined in schema\n", 
                                oc_row.cols[ 1 ], 0, 0 );
                        return LDAP_OTHER;      /* undefined objectClass ? */
                }
                
-               ber_str2bv( oc_row.cols[ 2 ], 0, 1, &oc_map->keytbl );
-               ber_str2bv( oc_row.cols[ 3 ], 0, 1, &oc_map->keycol );
-               oc_map->create_proc = ( oc_row.value_len[ 4 ] < 0 ) ? NULL 
+               ber_str2bv( oc_row.cols[ 2 ], 0, 1, &oc_map->bom_keytbl );
+               ber_str2bv( oc_row.cols[ 3 ], 0, 1, &oc_map->bom_keycol );
+               oc_map->bom_create_proc = ( oc_row.value_len[ 4 ] < 0 ) ? NULL 
                        : ch_strdup( oc_row.cols[ 4 ] );
 
                colnum = 5;
                if ( BACKSQL_CREATE_NEEDS_SELECT( si ) ) {
                        colnum = 6;
-                       oc_map->create_keyval = ( oc_row.value_len[ 5 ] < 0 ) 
+                       oc_map->bom_create_keyval = ( oc_row.value_len[ 5 ] < 0 ) 
                                ? NULL : ch_strdup( oc_row.cols[ 5 ] );
                }
-               oc_map->delete_proc = ( oc_row.value_len[ colnum ] < 0 ) ? NULL 
+               oc_map->bom_delete_proc = ( oc_row.value_len[ colnum ] < 0 ) ? NULL 
                        : ch_strdup( oc_row.cols[ colnum ] );
-               oc_map->expect_return = strtol( oc_row.cols[ colnum + 1 ], 
+               oc_map->bom_expect_return = strtol( oc_row.cols[ colnum + 1 ], 
                                NULL, 0 );
 
                /*
@@ -281,30 +315,40 @@ backsql_load_schema_map( backsql_info *si, SQLHDBC dbh )
                 * instructions in {create|delete}_proc
                 */
 
-               oc_map->attrs = NULL;
-               avl_insert( &si->oc_by_oc, oc_map, backsql_cmp_oc, NULL );
-               avl_insert( &si->oc_by_id, oc_map, backsql_cmp_oc_id, NULL );
-               oc_id = oc_map->id;
+               oc_map->bom_attrs = NULL;
+               if ( avl_insert( &si->oc_by_oc, oc_map, backsql_cmp_oc, avl_dup_error ) == -1 ) {
+                       Debug( LDAP_DEBUG_TRACE, "backsql_load_schema_map(): "
+                                       "duplicate objectClass \"%s\" in objectClass map\n",
+                                       oc_map->bom_oc->soc_cname.bv_val, 0, 0 );
+                       return LDAP_OTHER;
+               }
+               if ( avl_insert( &si->oc_by_id, oc_map, backsql_cmp_oc_id, avl_dup_error ) == -1 ) {
+                       Debug( LDAP_DEBUG_TRACE, "backsql_load_schema_map(): "
+                                       "duplicate objectClass \"%s\" in objectClass by ID map\n",
+                                       oc_map->bom_oc->soc_cname.bv_val, 0, 0 );
+                       return LDAP_OTHER;
+               }
+               oc_id = oc_map->bom_id;
                Debug( LDAP_DEBUG_TRACE, "load_schema_map(): "
                        "objectClass '%s': keytbl='%s' keycol='%s'\n",
                        BACKSQL_OC_NAME( oc_map ),
-                       oc_map->keytbl.bv_val, oc_map->keycol.bv_val );
-               if ( oc_map->create_proc ) {
+                       oc_map->bom_keytbl.bv_val, oc_map->bom_keycol.bv_val );
+               if ( oc_map->bom_create_proc ) {
                        Debug( LDAP_DEBUG_TRACE, "create_proc='%s'\n",
-                               oc_map->create_proc, 0, 0 );
+                               oc_map->bom_create_proc, 0, 0 );
                }
-               if ( oc_map->create_keyval ) {
+               if ( oc_map->bom_create_keyval ) {
                        Debug( LDAP_DEBUG_TRACE, "create_keyval='%s'\n",
-                               oc_map->create_keyval, 0, 0 );
+                               oc_map->bom_create_keyval, 0, 0 );
                }
-               if ( oc_map->delete_proc ) {
+               if ( oc_map->bom_delete_proc ) {
                        Debug( LDAP_DEBUG_TRACE, "delete_proc='%s'\n", 
-                               oc_map->delete_proc, 0, 0 );
+                               oc_map->bom_delete_proc, 0, 0 );
                }
                Debug( LDAP_DEBUG_TRACE, "expect_return: "
                        "add=%d, del=%d; attributes:\n",
-                       BACKSQL_IS_ADD( oc_map->expect_return ), 
-                       BACKSQL_IS_DEL( oc_map->expect_return ), 0 );
+                       BACKSQL_IS_ADD( oc_map->bom_expect_return ), 
+                       BACKSQL_IS_DEL( oc_map->bom_expect_return ), 0 );
 
                Debug( LDAP_DEBUG_TRACE, "load_schema_map(): "
                        "autoadding 'objectClass' and 'ref' mappings\n",
@@ -342,7 +386,7 @@ backsql_load_schema_map( backsql_info *si, SQLHDBC dbh )
                        at_map = (backsql_at_map_rec *)ch_calloc( 1,
                                        sizeof( backsql_at_map_rec ) );
                        rc = slap_str2ad( at_row.cols[ 0 ], 
-                                       &at_map->ad, &text );
+                                       &at_map->bam_ad, &text );
                        if ( rc != LDAP_SUCCESS ) {
                                Debug( LDAP_DEBUG_TRACE, "load_schema_map(): "
                                        "attribute '%s' for objectClass '%s' "
@@ -351,44 +395,50 @@ backsql_load_schema_map( backsql_info *si, SQLHDBC dbh )
                                        BACKSQL_OC_NAME( oc_map ), text );
                                return LDAP_CONSTRAINT_VIOLATION;
                        }
-                               
-                       ber_str2bv( at_row.cols[ 1 ], 0, 1, &at_map->sel_expr );
+
+                       ber_str2bv( at_row.cols[ 1 ], 0, 1, &at_map->bam_sel_expr );
                        if ( at_row.value_len[ 8 ] < 0 ) {
-                               at_map->sel_expr_u.bv_val = NULL;
-                               at_map->sel_expr_u.bv_len = 0;
+                               at_map->bam_sel_expr_u.bv_val = NULL;
+                               at_map->bam_sel_expr_u.bv_len = 0;
                        } else {
                                ber_str2bv( at_row.cols[ 8 ], 0, 1, 
-                                               &at_map->sel_expr_u );
+                                               &at_map->bam_sel_expr_u );
                        }
 
                        ber_str2bv( at_row.cols[ 2 ], 0, 0, &bv );
                        backsql_merge_from_clause( &bb, &bv );
-                       at_map->from_tbls = bb.bb_val;
+                       at_map->bam_from_tbls = bb.bb_val;
                        if ( at_row.value_len[ 3 ] < 0 ) {
-                               at_map->join_where.bv_val = NULL;
-                               at_map->join_where.bv_len = 0;
+                               at_map->bam_join_where.bv_val = NULL;
+                               at_map->bam_join_where.bv_len = 0;
                        } else {
                                ber_str2bv( at_row.cols[ 3 ], 0, 1, 
-                                               &at_map->join_where );
+                                               &at_map->bam_join_where );
                        }
-                       at_map->add_proc = NULL;
+                       at_map->bam_add_proc = NULL;
                        if ( at_row.value_len[ 4 ] > 0 ) {
-                               at_map->add_proc = ch_strdup( at_row.cols[4] );
+                               at_map->bam_add_proc = ch_strdup( at_row.cols[4] );
                        }
-                       at_map->delete_proc = NULL;
+                       at_map->bam_delete_proc = NULL;
                        if ( at_row.value_len[ 5 ] > 0 ) {
-                               at_map->delete_proc
-                                       = ch_strdup( at_row.cols[ 5 ] );
+                               at_map->bam_delete_proc = ch_strdup( at_row.cols[ 5 ] );
                        }
-                       at_map->param_order = strtol( at_row.cols[ 6 ], 
+                       at_map->bam_param_order = strtol( at_row.cols[ 6 ], 
                                        NULL, 0 );
-                       at_map->expect_return = strtol( at_row.cols[ 7 ],
+                       at_map->bam_expect_return = strtol( at_row.cols[ 7 ],
                                        NULL, 0 );
                        backsql_make_attr_query( oc_map, at_map );
                        Debug( LDAP_DEBUG_TRACE, "load_schema_map(): "
                                "preconstructed query '%s'\n",
-                               at_map->query, 0, 0 );
-                       avl_insert( &oc_map->attrs, at_map, backsql_cmp_attr, NULL );
+                               at_map->bam_query, 0, 0 );
+                       at_map->bam_next = NULL;
+                       if ( avl_insert( &oc_map->bom_attrs, at_map, backsql_cmp_attr, backsql_dup_attr ) == BACKSQL_DUPLICATE ) {
+                               Debug( LDAP_DEBUG_TRACE, "backsql_load_schema_map(): "
+                                               "duplicate attribute \"%s\" "
+                                               "in objectClass \"%s\" map\n",
+                                               at_map->bam_ad->ad_cname.bv_val,
+                                               oc_map->bom_oc->soc_cname.bv_val, 0 );
+                       }
                }
                backsql_FreeRow( &at_row );
                SQLFreeStmt( at_sth, SQL_CLOSE );
@@ -412,7 +462,7 @@ backsql_oc2oc( backsql_info *si, ObjectClass *oc )
                objclass, 0, 0 );
 #endif /* BACKSQL_TRACE */
 
-       tmp.oc = oc;
+       tmp.bom_oc = oc;
        res = (backsql_oc_map_rec *)avl_find( si->oc_by_oc, &tmp, backsql_cmp_oc );
 #ifdef BACKSQL_TRACE
        if ( res != NULL ) {
@@ -439,8 +489,8 @@ backsql_name2oc( backsql_info *si, struct berval *oc_name )
                objclass, 0, 0 );
 #endif /* BACKSQL_TRACE */
 
-       tmp.oc = oc_bvfind( oc_name );
-       if ( tmp.oc == NULL ) {
+       tmp.bom_oc = oc_bvfind( oc_name );
+       if ( tmp.bom_oc == NULL ) {
                return NULL;
        }
 
@@ -449,7 +499,7 @@ backsql_name2oc( backsql_info *si, struct berval *oc_name )
        if ( res != NULL ) {
                Debug( LDAP_DEBUG_TRACE, "<==oc_with_name(): "
                        "found name='%s', id=%d\n", 
-                       BACKSQL_OC_NAME( res ), res->id, 0 );
+                       BACKSQL_OC_NAME( res ), res->bom_id, 0 );
        } else {
                Debug( LDAP_DEBUG_TRACE, "<==oc_with_name(): "
                        "not found\n", 0, 0, 0 );
@@ -469,7 +519,7 @@ backsql_id2oc( backsql_info *si, unsigned long id )
                "searching for objectclass with id='%d'\n", id, 0, 0 );
 #endif /* BACKSQL_TRACE */
 
-       tmp.id = id;
+       tmp.bom_id = id;
        res = (backsql_oc_map_rec *)avl_find( si->oc_by_id, &tmp,
                        backsql_cmp_oc_id );
 
@@ -477,7 +527,7 @@ backsql_id2oc( backsql_info *si, unsigned long id )
        if ( res != NULL ) {
                Debug( LDAP_DEBUG_TRACE, "<==oc_with_name(): "
                        "found name='%s', id=%d\n",
-                       BACKSQL_OC_NAME( res ), res->id, 0 );
+                       BACKSQL_OC_NAME( res ), res->bom_id, 0 );
        } else {
                Debug( LDAP_DEBUG_TRACE, "<==oc_with_name(): "
                        "not found\n", 0, 0, 0 );
@@ -498,15 +548,16 @@ backsql_ad2at( backsql_oc_map_rec* objclass, AttributeDescription *ad )
                attr, BACKSQL_OC_NAME( objclass ), 0 );
 #endif /* BACKSQL_TRACE */
 
-       tmp.ad = ad;
-       res = (backsql_at_map_rec *)avl_find( objclass->attrs, &tmp,
+       tmp.bam_ad = ad;
+       res = (backsql_at_map_rec *)avl_find( objclass->bom_attrs, &tmp,
                        backsql_cmp_attr );
 
 #ifdef BACKSQL_TRACE
        if ( res != NULL ) {
                Debug( LDAP_DEBUG_TRACE, "<==backsql_ad2at(): "
                        "found name='%s', sel_expr='%s'\n",
-                       res->ad->ad_cname.bv_val, res->sel_expr.bv_val, 0 );
+                       res->bam_ad->ad_cname.bv_val,
+                       res->bam_sel_expr.bv_val, 0 );
        } else {
                Debug( LDAP_DEBUG_TRACE, "<==backsql_ad2at(): "
                        "not found\n", 0, 0, 0 );
@@ -516,68 +567,103 @@ backsql_ad2at( backsql_oc_map_rec* objclass, AttributeDescription *ad )
        return res;
 }
 
+/* attributeType inheritance */
+struct supad2at_t {
+       backsql_at_map_rec      **ret;
+       AttributeDescription    *ad;
+       unsigned                n;
+};
+
+#define SUPAD2AT_STOP  (-1)
+
+static int
+supad2at_f( void *v_at, void *v_arg )
+{
+       backsql_at_map_rec      *at = (backsql_at_map_rec *)v_at;
+       struct supad2at_t       *va = (struct supad2at_t *)v_arg;
+
+       if ( is_at_subtype( at->bam_ad->ad_type, va->ad->ad_type ) ) {
+               backsql_at_map_rec      **ret;
+
+               ret = ch_realloc( va->ret, sizeof( backsql_at_map_rec *) * ( va->n + 2 ) );
+               if ( ret == NULL ) {
+                       ch_free( va->ret );
+                       return SUPAD2AT_STOP;
+               }
+
+               ret[ va->n ] = at;
+               va->n++;
+               ret[ va->n ] = NULL;
+               va->ret = ret;
+       }
+
+       return 0;
+}
+
 /*
- * Deprecated
+ * stores in *pret a NULL terminated array of pointers
+ * to backsql_at_map_rec whose attributeType is supad->ad_type 
+ * or derived from it
  */
-backsql_at_map_rec *
-backsql_name2at( backsql_oc_map_rec* objclass, struct berval *attr )
+int
+backsql_supad2at( backsql_oc_map_rec *objclass, AttributeDescription *supad,
+               backsql_at_map_rec ***pret )
 {
-       backsql_at_map_rec      tmp, *res;
-       const char              *text = NULL;
-#ifdef BACKSQL_TRACE
-       Debug( LDAP_DEBUG_TRACE, "==>backsql_name2at(): "
-               "searching for attribute '%s' for objectclass '%s'\n",
-               attr, BACKSQL_OC_NAME( objclass ), 0 );
-#endif /* BACKSQL_TRACE */
+       struct supad2at_t       va;
+       int                     rc;
 
-       if ( slap_bv2ad( attr, &tmp.ad, &text ) != LDAP_SUCCESS ) {
-               return NULL;
-       }
+       assert( objclass );
+       assert( supad );
+       assert( pret );
 
-       res = (backsql_at_map_rec *)avl_find( objclass->attrs, &tmp,
-                       backsql_cmp_attr );
+       *pret = NULL;
 
-#ifdef BACKSQL_TRACE
-       if ( res != NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "<==backsql_name2at(): "
-                       "found name='%s', sel_expr='%s'\n",
-                       res->name, res->sel_expr.bv_val, 0 );
-       } else {
-               Debug( LDAP_DEBUG_TRACE, "<==backsql_name2at(): "
-                       "not found\n", 0, 0, 0 );
+       va.ret = NULL;
+       va.ad = supad;
+       va.n = 0;
+       
+       rc = avl_apply( objclass->bom_attrs, supad2at_f, &va,
+                       SUPAD2AT_STOP, AVL_INORDER );
+       if ( rc == SUPAD2AT_STOP ) {
+               return -1;
        }
-#endif /* BACKSQL_TRACE */
 
-       return res;
+       *pret = va.ret;
+
+       return 0;
 }
 
 static void
 backsql_free_attr( void *v_at )
 {
-       backsql_at_map_rec *at = v_at;
+       backsql_at_map_rec      *at = v_at;
+       
        Debug( LDAP_DEBUG_TRACE, "==>free_attr(): '%s'\n", 
-                       at->ad->ad_cname.bv_val, 0, 0 );
-       ch_free( at->sel_expr.bv_val );
-       if ( at->from_tbls.bv_val != NULL ) {
-               ch_free( at->from_tbls.bv_val );
+                       at->bam_ad->ad_cname.bv_val, 0, 0 );
+       ch_free( at->bam_sel_expr.bv_val );
+       if ( at->bam_from_tbls.bv_val != NULL ) {
+               ch_free( at->bam_from_tbls.bv_val );
        }
-       if ( at->join_where.bv_val != NULL ) {
-               ch_free( at->join_where.bv_val );
+       if ( at->bam_join_where.bv_val != NULL ) {
+               ch_free( at->bam_join_where.bv_val );
        }
-       if ( at->add_proc != NULL ) {
-               ch_free( at->add_proc );
+       if ( at->bam_add_proc != NULL ) {
+               ch_free( at->bam_add_proc );
        }
-       if ( at->delete_proc != NULL ) {
-               ch_free( at->delete_proc );
+       if ( at->bam_delete_proc != NULL ) {
+               ch_free( at->bam_delete_proc );
        }
-       if ( at->query ) {
-               ch_free( at->query );
+       if ( at->bam_query ) {
+               ch_free( at->bam_query );
        }
 
        /* TimesTen */
-       if ( at->sel_expr_u.bv_val ) {
-               ch_free( at->sel_expr_u.bv_val );
+       if ( at->bam_sel_expr_u.bv_val ) {
+               ch_free( at->bam_sel_expr_u.bv_val );
+       }
+
+       if ( at->bam_next ) {
+               backsql_free_attr( at->bam_next );
        }
        
        ch_free( at );
@@ -588,20 +674,21 @@ backsql_free_attr( void *v_at )
 static void
 backsql_free_oc( void *v_oc )
 {
-       backsql_oc_map_rec *oc = v_oc;
+       backsql_oc_map_rec      *oc = v_oc;
+       
        Debug( LDAP_DEBUG_TRACE, "==>free_oc(): '%s'\n", 
                        BACKSQL_OC_NAME( oc ), 0, 0 );
-       avl_free( oc->attrs, backsql_free_attr );
-       ch_free( oc->keytbl.bv_val );
-       ch_free( oc->keycol.bv_val );
-       if ( oc->create_proc != NULL ) {
-               ch_free( oc->create_proc );
+       avl_free( oc->bom_attrs, backsql_free_attr );
+       ch_free( oc->bom_keytbl.bv_val );
+       ch_free( oc->bom_keycol.bv_val );
+       if ( oc->bom_create_proc != NULL ) {
+               ch_free( oc->bom_create_proc );
        }
-       if ( oc->create_keyval != NULL ) {
-               ch_free( oc->create_keyval );
+       if ( oc->bom_create_keyval != NULL ) {
+               ch_free( oc->bom_create_keyval );
        }
-       if ( oc->delete_proc != NULL ) {
-               ch_free( oc->delete_proc );
+       if ( oc->bom_delete_proc != NULL ) {
+               ch_free( oc->bom_delete_proc );
        }
        ch_free( oc );