]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/backover.c
Always exclude subordinates from top-level ops so glue overlay can
[openldap] / servers / slapd / backover.c
index d51de28535a485b5ec59ca3744ee68d820e1df24..c3fe17e601a94e1dbc8920e86c40c5fb85a5be3e 100644 (file)
@@ -36,8 +36,9 @@ over_db_func(
        enum db_which which
 )
 {
-       slap_overinfo *oi = (slap_overinfo *) be->bd_info;
+       slap_overinfo *oi = be->bd_info->bi_private;
        slap_overinst *on = oi->oi_list;
+       BackendInfo *bi_orig = be->bd_info;
        BI_db_open **func;
        int rc = 0;
 
@@ -54,7 +55,7 @@ over_db_func(
                        rc = func[which]( be );
                }
        }
-       be->bd_info = (BackendInfo *)oi;
+       be->bd_info = bi_orig;
        return rc;
 }
 
@@ -67,14 +68,57 @@ over_db_config(
        char **argv
 )
 {
-       slap_overinfo *oi = (slap_overinfo *) be->bd_info;
+       slap_overinfo *oi = be->bd_info->bi_private;
        slap_overinst *on = oi->oi_list;
+       BackendInfo *bi_orig = be->bd_info;
        int rc = 0;
 
        if ( oi->oi_orig->bi_db_config ) {
                be->bd_info = oi->oi_orig;
                rc = oi->oi_orig->bi_db_config( be, fname, lineno,
                        argc, argv );
+
+               if ( be->bd_info != oi->oi_orig ) {
+                       slap_overinfo   *oi2;
+                       slap_overinst   *on2, **onp;
+                       BackendDB       be2 = *be;
+                       int             i;
+
+                       /* a database added an overlay;
+                        * work it around... */
+                       assert( overlay_is_over( be ) );
+                       
+                       oi2 = ( slap_overinfo * )be->bd_info->bi_private;
+                       on2 = oi2->oi_list;
+
+                       /* need to put a uniqueness check here as well;
+                        * note that in principle there could be more than
+                        * one overlay as a result of multiple calls to
+                        * overlay_config() */
+                       be2.bd_info = (BackendInfo *)oi;
+
+                       for ( i = 0, onp = &on2; *onp; i++, onp = &(*onp)->on_next ) {
+                               if ( overlay_is_inst( &be2, (*onp)->on_bi.bi_type ) ) {
+                                       Debug( LDAP_DEBUG_ANY, "over_db_config(): "
+                                                       "warning, freshly added "
+                                                       "overlay #%d \"%s\" is already in list\n",
+                                                       i, (*onp)->on_bi.bi_type, 0 );
+
+                                       /* NOTE: if the overlay already exists,
+                                        * there is no way to merge the results
+                                        * of the configuration that may have 
+                                        * occurred during bi_db_config(); we
+                                        * just issue a warning, and the 
+                                        * administrator should deal with this */
+                               }
+                       }
+                       *onp = oi->oi_list;
+
+                       oi->oi_list = on2;
+
+                       ch_free( be->bd_info );
+               }
+
                be->bd_info = (BackendInfo *)oi;
                if ( rc != SLAP_CONF_UNKNOWN ) return rc;
        }
@@ -87,7 +131,7 @@ over_db_config(
                        if ( rc != SLAP_CONF_UNKNOWN ) break;
                }
        }
-       be->bd_info = (BackendInfo *)oi;
+       be->bd_info = bi_orig;
        return rc;
 }
 
@@ -104,7 +148,25 @@ over_db_close(
        BackendDB *be
 )
 {
-       return over_db_func( be, db_close );
+       slap_overinfo *oi = be->bd_info->bi_private;
+       slap_overinst *on = oi->oi_list;
+       BackendInfo *bi_orig = be->bd_info;
+       int rc = 0;
+
+       for (; on && rc == 0; on=on->on_next) {
+               be->bd_info = &on->on_bi;
+               if ( be->bd_info->bi_db_close ) {
+                       rc = be->bd_info->bi_db_close( be );
+               }
+       }
+
+       if ( oi->oi_orig->bi_db_close ) {
+               be->bd_info = oi->oi_orig;
+               rc = be->bd_info->bi_db_close( be );
+       }
+
+       be->bd_info = bi_orig;
+       return rc;
 }
 
 static int
@@ -112,7 +174,7 @@ over_db_destroy(
        BackendDB *be
 )
 {
-       slap_overinfo *oi = (slap_overinfo *) be->bd_info;
+       slap_overinfo *oi = be->bd_info->bi_private;
        slap_overinst *on = oi->oi_list, *next;
        int rc;
 
@@ -134,6 +196,7 @@ over_back_response ( Operation *op, SlapReply *rs )
        int rc = SLAP_CB_CONTINUE;
        BackendDB *be = op->o_bd, db = *op->o_bd;
 
+       db.be_flags |= SLAP_DBFLAG_OVERLAY;
        op->o_bd = &db;
        for (; on; on=on->on_next ) {
                if ( on->on_response ) {
@@ -158,6 +221,7 @@ enum op_which {
        op_abandon,
        op_cancel,
        op_extended,
+       op_aux_operational,
        op_aux_chk_referrals,
        op_last
 };
@@ -178,6 +242,7 @@ static int op_rc[] = {
        LDAP_UNWILLING_TO_PERFORM,      /* abandon */
        LDAP_UNWILLING_TO_PERFORM,      /* cancel */
        LDAP_UNWILLING_TO_PERFORM,      /* extended */
+       LDAP_SUCCESS,                   /* aux_operational */
        LDAP_SUCCESS                    /* aux_chk_referrals */
 };
 
@@ -188,14 +253,18 @@ over_op_func(
        enum op_which which
 )
 {
-       slap_overinfo *oi = (slap_overinfo *) op->o_bd->bd_info;
+       slap_overinfo *oi = op->o_bd->bd_info->bi_private;
        slap_overinst *on = oi->oi_list;
        BI_op_bind **func;
-       BackendDB *be = op->o_bd, db = *op->o_bd;
+       BackendDB *be = op->o_bd, db;
        slap_callback cb = {NULL, over_back_response, NULL, NULL};
        int rc = SLAP_CB_CONTINUE;
 
-       op->o_bd = &db;
+       if ( !SLAP_ISOVERLAY( op->o_bd )) {
+               db = *op->o_bd;
+               db.be_flags |= SLAP_DBFLAG_OVERLAY;
+               op->o_bd = &db;
+       }
        cb.sc_next = op->o_callback;
        cb.sc_private = oi;
        op->o_callback = &cb;
@@ -209,15 +278,16 @@ over_op_func(
                }
        }
 
-       op->o_bd = be;
        func = &oi->oi_orig->bi_op_bind;
        if ( func[which] && rc == SLAP_CB_CONTINUE ) {
+               db.bd_info = oi->oi_orig;
                rc = func[which]( op, rs );
        }
        /* should not fall thru this far without anything happening... */
        if ( rc == SLAP_CB_CONTINUE ) {
                rc = op_rc[ which ];
        }
+       op->o_bd = be;
        op->o_callback = cb.sc_next;
        return rc;
 }
@@ -289,7 +359,13 @@ over_op_extended( Operation *op, SlapReply *rs )
 }
 
 static int
-over_chk_referrals( Operation *op, SlapReply *rs )
+over_aux_operational( Operation *op, SlapReply *rs )
+{
+       return over_op_func( op, rs, op_aux_operational );
+}
+
+static int
+over_aux_chk_referrals( Operation *op, SlapReply *rs )
 {
        return over_op_func( op, rs, op_aux_chk_referrals );
 }
@@ -304,6 +380,13 @@ overlay_register(
        return 0;
 }
 
+/*
+ * iterator on registered overlays; overlay_next( NULL ) returns the first
+ * overlay; * subsequent calls with the previously returned value allow to 
+ * iterate * over the entire list; returns NULL when no more overlays are 
+ * registered.
+ */
+
 slap_overinst *
 overlay_next(
        slap_overinst *on
@@ -316,32 +399,94 @@ overlay_next(
        return on->on_next;
 }
 
+/*
+ * returns a specific registered overlay based on the type; NULL if not
+ * registered.
+ */
+
+slap_overinst *
+overlay_find( const char *over_type )
+{
+       slap_overinst *on = overlays;
+
+       assert( over_type );
+
+       for ( ; on; on = on->on_next ) {
+               if ( strcmp( on->on_bi.bi_type, over_type ) == 0 ) {
+                       break;
+               }
+       }
+
+       return on;
+}
+
 static const char overtype[] = "over";
 
+/*
+ * returns TRUE (1) if the database is actually an overlay instance;
+ * FALSE (0) otherwise.
+ */
+
+int
+overlay_is_over( BackendDB *be )
+{
+       return be->bd_info->bi_type == overtype;
+}
+
+/*
+ * returns TRUE (1) if the given database is actually an overlay
+ * instance and, somewhere in the list, contains the requested overlay;
+ * FALSE (0) otherwise.
+ */
+
+int
+overlay_is_inst( BackendDB *be, const char *over_type )
+{
+       slap_overinst   *on;
+
+       assert( be );
+
+       if ( !overlay_is_over( be ) ) {
+               return 0;
+       }
+       
+       on = ((slap_overinfo *)be->bd_info->bi_private)->oi_list;
+       for ( ; on; on = on->on_next ) {
+               if ( strcmp( on->on_bi.bi_type, over_type ) == 0 ) {
+                       return 1;
+               }
+       }
+
+       return 0;
+}
+
 /* add an overlay to a particular backend. */
 int
 overlay_config( BackendDB *be, const char *ov )
 {
-       slap_overinst *on = NULL, *on2 = NULL, *prev = NULL;
+       slap_overinst *on = NULL, *on2 = NULL;
        slap_overinfo *oi = NULL;
        BackendInfo *bi = NULL;
 
-       for ( on = overlays; on; on=on->on_next ) {
-               if (!strcmp( ov, on->on_bi.bi_type ) )
-                       break;
-       }
-       if (!on) {
-               Debug( LDAP_DEBUG_ANY, "overlay %s not found\n", ov, 0, 0 );
+       on = overlay_find( ov );
+       if ( !on ) {
+               Debug( LDAP_DEBUG_ANY, "overlay \"%s\" not found\n", ov, 0, 0 );
                return 1;
        }
 
        /* If this is the first overlay on this backend, set up the
         * overlay info structure
         */
-       if ( be->bd_info->bi_type != overtype ) {
-               oi = ch_malloc( sizeof(slap_overinfo) );
+       if ( !overlay_is_over( be ) ) {
+               oi = ch_malloc( sizeof( slap_overinfo ) );
                oi->oi_orig = be->bd_info;
                oi->oi_bi = *be->bd_info;
+
+               /* Save a pointer to ourself in bi_private.
+                * This allows us to keep working in conjunction
+                * with backglue...
+                */
+               oi->oi_bi.bi_private = oi;
                oi->oi_list = NULL;
                bi = (BackendInfo *)oi;
 
@@ -371,29 +516,21 @@ overlay_config( BackendDB *be, const char *ov )
                 * all the hooks to share the same args
                 * of the operations...
                 */
-               bi->bi_chk_referrals = over_chk_referrals;
+               bi->bi_operational = over_aux_operational;
+               bi->bi_chk_referrals = over_aux_chk_referrals;
 
                be->bd_info = bi;
 
        } else {
-               oi = (slap_overinfo *) be->bd_info;
-       }
+               if ( overlay_is_inst( be, ov ) ) {
+                       Debug( LDAP_DEBUG_ANY, "overlay_config(): "
+                                       "warning, overlay \"%s\" "
+                                       "already in list\n", ov, 0, 0 );
+               }
 
-#if 0
-       /* Walk to the end of the list of overlays, add the new
-        * one onto the end
-        */
-       for ( prev=NULL, on2 = oi->oi_list; on2; prev=on2, on2=on2->on_next );
-       on2 = ch_calloc( 1, sizeof(slap_overinst) );
-       if ( !prev ) {
-               oi->oi_list = on2;
-       } else {
-               prev->on_next = on2;
+               oi = be->bd_info->bi_private;
        }
-       *on2 = *on;
-       on2->on_next = NULL;
-       on2->on_info = oi;
-#else
+
        /* Insert new overlay on head of list. Overlays are executed
         * in reverse of config order...
         */
@@ -402,7 +539,6 @@ overlay_config( BackendDB *be, const char *ov )
        on2->on_info = oi;
        on2->on_next = oi->oi_list;
        oi->oi_list = on2;
-#endif
 
        /* Any initialization needed? */
        if ( on->on_bi.bi_db_init ) {