]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/bconfig.c
Fix C errors
[openldap] / servers / slapd / bconfig.c
index 9ec4b7f10ba52a0ebb69c38179206d8ada26aba3..c01ec584a51ef30757316ade1c5a0b337d663cc4 100644 (file)
@@ -555,6 +555,7 @@ static ConfigTable config_back_cf_table[] = {
                        "SYNTAX OMsDN )", NULL, NULL },
        { "syncrepl", NULL, 0, 0, 0, ARG_DB|ARG_MAGIC,
                &syncrepl_config, "( OLcfgDbAt:0.11 NAME 'olcSyncrepl' "
+                       "EQUALITY caseIgnoreMatch "
                        "SYNTAX OMsDirectoryString X-ORDERED 'VALUES' )", NULL, NULL },
        { "threads", "count", 2, 2, 0,
 #ifdef NO_THREADS
@@ -1602,7 +1603,7 @@ config_generic(ConfigArgs *c) {
                                        si = ch_malloc( sizeof(ServerID) + len + 1 );
                                        si->si_url.bv_val = (char *)(si+1);
                                        si->si_url.bv_len = len;
-                                       strcpy( si->si_url.bv_val, c->argv[3] );
+                                       strcpy( si->si_url.bv_val, c->argv[2] );
                                } else {
                                        if ( sid_list ) {
                                                snprintf( c->msg, sizeof( c->msg ),
@@ -3870,6 +3871,8 @@ config_rename_one( Operation *op, SlapReply *rs, Entry *e,
                op->o_callback = ≻
                op->orr_newrdn = *newrdn;
                op->orr_nnewrdn = *nnewrdn;
+               op->orr_newSup = NULL;
+               op->orr_nnewSup = NULL;
                op->orr_deleteoldrdn = 1;
                op->orr_modlist = NULL;
                slap_modrdn2mods( op, rs );
@@ -3993,6 +3996,10 @@ check_name_index( CfEntryInfo *parent, ConfigType ce_type, Entry *e,
                if ( ce->ce_type == ce_type ) nsibs++;
        }
 
+       /* account for -1 frontend */
+       if ( ce_type == Cft_Database )
+               nsibs--;
+
        if ( index != nsibs ) {
                if ( gotindex ) {
                        if ( index < nsibs ) {
@@ -4002,8 +4009,9 @@ check_name_index( CfEntryInfo *parent, ConfigType ce_type, Entry *e,
                                        renumber = 1;
                        }
                }
-               if ( !isfrontend && index == -1 )
+               if ( !isfrontend && index == -1 ) {
                        index = nsibs;
+               }
 
                /* just make index = nsibs */
                if ( !renumber ) {
@@ -4100,6 +4108,50 @@ cfAddOverlay( CfEntryInfo *p, Entry *e, struct config_args_s *ca )
        return LDAP_SUCCESS;
 }
 
+static void
+schema_destroy_one( ConfigArgs *ca, ConfigOCs **colst, int nocs,
+       CfEntryInfo *p )
+{
+       ConfigTable *ct;
+       ConfigFile *cfo;
+       AttributeDescription *ad;
+       const char *text;
+
+       ca->valx = -1;
+       ca->line = NULL;
+       if ( cfn->c_cr_head ) {
+               struct berval bv = BER_BVC("olcDitContentRules");
+               ad = NULL;
+               slap_bv2ad( &bv, &ad, &text );
+               ct = config_find_table( colst, nocs, ad );
+               config_del_vals( ct, ca );
+       }
+       if ( cfn->c_oc_head ) {
+               struct berval bv = BER_BVC("olcObjectClasses");
+               ad = NULL;
+               slap_bv2ad( &bv, &ad, &text );
+               ct = config_find_table( colst, nocs, ad );
+               config_del_vals( ct, ca );
+       }
+       if ( cfn->c_at_head ) {
+               struct berval bv = BER_BVC("olcAttributeTypes");
+               ad = NULL;
+               slap_bv2ad( &bv, &ad, &text );
+               ct = config_find_table( colst, nocs, ad );
+               config_del_vals( ct, ca );
+       }
+       if ( cfn->c_om_head ) {
+               struct berval bv = BER_BVC("olcObjectIdentifier");
+               ad = NULL;
+               slap_bv2ad( &bv, &ad, &text );
+               ct = config_find_table( colst, nocs, ad );
+               config_del_vals( ct, ca );
+       }
+       cfo = p->ce_private;
+       cfo->c_kids = cfn->c_sibs;
+       ch_free( cfn );
+}
+
 /* Parse an LDAP entry into config directives */
 static int
 config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
@@ -4113,6 +4165,8 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
        ConfigTable *ct;
        char *ptr;
 
+       memset( ca, 0, sizeof(ConfigArgs));
+
        /* Make sure parent exists and entry does not. But allow
         * Databases and Overlays to be inserted. Don't do any
         * auto-renumbering if manageDSAit control is present.
@@ -4148,8 +4202,6 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
        oc_at = attr_find( e->e_attrs, slap_schema.si_ad_objectClass );
        if ( !oc_at ) return LDAP_OBJECT_CLASS_VIOLATION;
 
-       memset( ca, 0, sizeof(ConfigArgs));
-
        /* Fake the coordinates based on whether we're part of an
         * LDAP Add or if reading the config dir
         */
@@ -4195,7 +4247,7 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
        }
 
        if ( rc != LDAP_SUCCESS )
-               goto done;
+               goto done_noop;
 
        /* Parse all the values and check for simple syntax errors before
         * performing any set actions.
@@ -4217,14 +4269,14 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
                rc = check_name_index( last, colst[0]->co_type, e, rs, renum,
                        &ibase );
                if ( rc ) {
-                       goto done;
+                       goto done_noop;
                }
                if ( renum && *renum && colst[0]->co_type != Cft_Database &&
                        colst[0]->co_type != Cft_Overlay ) {
                        snprintf( ca->msg, sizeof( ca->msg ),
                                "operation requires sibling renumbering" );
                        rc = LDAP_UNWILLING_TO_PERFORM;
-                       goto done;
+                       goto done_noop;
                }
        }
 
@@ -4238,7 +4290,7 @@ config_add_internal( CfBackInfo *cfb, Entry *e, ConfigArgs *ca, SlapReply *rs,
                ct = config_find_table( colst, nocs, a->a_desc );
                if ( !ct ) continue;    /* user data? */
                rc = check_vals( ct, ca, a, 1 );
-               if ( rc ) goto done;
+               if ( rc ) goto done_noop;
        }
 
        /* Basic syntax checks are OK. Do the actual settings. */
@@ -4350,8 +4402,11 @@ done:
                                backend_destroy_one( ca->be, 1 );
                } else if ( (colst[0]->co_type == Cft_Overlay) && ca->bi ) {
                        overlay_destroy_one( ca->be, (slap_overinst *)ca->bi );
+               } else if ( colst[0]->co_type == Cft_Schema ) {
+                       schema_destroy_one( ca, colst, nocs, last );
                }
        }
+done_noop:
 
        ch_free( ca->argv );
        if ( colst ) ch_free( colst );
@@ -4527,6 +4582,33 @@ typedef struct delrec {
        int idx[1];
 } delrec;
 
+static int
+config_modify_add( ConfigTable *ct, ConfigArgs *ca, AttributeDescription *ad,
+       int i )
+{
+       int rc;
+
+       if (ad->ad_type->sat_flags & SLAP_AT_ORDERED &&
+               ca->line[0] == '{' )
+       {
+               char *ptr = strchr( ca->line + 1, '}' );
+               if ( ptr ) {
+                       char    *next;
+
+                       ca->valx = strtol( ca->line + 1, &next, 0 );
+                       if ( next == ca->line + 1 || next[ 0 ] != '}' ) {
+                               return LDAP_OTHER;
+                       }
+                       ca->line = ptr+1;
+               }
+       }
+       rc = config_parse_add( ct, ca, i );
+       if ( rc ) {
+               rc = LDAP_OTHER;
+       }
+       return rc;
+}
+
 static int
 config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
        ConfigArgs *ca )
@@ -4534,7 +4616,7 @@ config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
        int rc = LDAP_UNWILLING_TO_PERFORM;
        Modifications *ml;
        Entry *e = ce->ce_entry;
-       Attribute *save_attrs = e->e_attrs, *oc_at;
+       Attribute *save_attrs = e->e_attrs, *oc_at, *s, *a;
        ConfigTable *ct;
        ConfigOCs **colst;
        int i, nocs;
@@ -4546,6 +4628,11 @@ config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
 
        colst = count_ocs( oc_at, &nocs );
 
+       /* make sure add/del flags are clear; should always be true */
+       for ( s = save_attrs; s; s = s->a_next ) {
+               s->a_flags &= ~(SLAP_ATTR_IXADD|SLAP_ATTR_IXDEL);
+       }
+
        e->e_attrs = attrs_dup( e->e_attrs );
 
        init_config_argv( ca );
@@ -4568,7 +4655,7 @@ config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
                                rc = LDAP_OTHER;
                                snprintf(ca->msg, sizeof(ca->msg), "cannot delete %s",
                                        ml->sml_desc->ad_cname.bv_val );
-                               goto out;
+                               goto out_noop;
                        }
                        if ( ml->sml_op == LDAP_MOD_REPLACE ) {
                                vals = ml->sml_values;
@@ -4630,11 +4717,11 @@ config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
                                                        rc = LDAP_OTHER;
                                                        snprintf(ca->msg, sizeof(ca->msg), "cannot insert %s",
                                                                ml->sml_desc->ad_cname.bv_val );
-                                                       goto out;
+                                                       goto out_noop;
                                                }
                                        }
                                        rc = check_vals( ct, ca, ml, 0 );
-                                       if ( rc ) goto out;
+                                       if ( rc ) goto out_noop;
                                }
                        }
                        rc = modify_add_values(e, &ml->sml_mod,
@@ -4663,117 +4750,144 @@ config_modify_internal( CfEntryInfo *ce, Operation *op, SlapReply *rs,
                if(rc != LDAP_SUCCESS) break;
        }
        
-       if(rc == LDAP_SUCCESS) {
+       if ( rc == LDAP_SUCCESS) {
                /* check that the entry still obeys the schema */
                rc = entry_schema_check(op, e, NULL, 0, 0,
                        &rs->sr_text, ca->msg, sizeof(ca->msg) );
+               if ( rc ) goto out_noop;
        }
-       if ( rc == LDAP_SUCCESS ) {
-               /* Basic syntax checks are OK. Do the actual settings. */
-               for ( ml = op->orm_modlist; ml; ml = ml->sml_next ) {
-                       ct = config_find_table( colst, nocs, ml->sml_desc );
-                       if ( !ct ) continue;
-
-                       switch (ml->sml_op) {
-                       case LDAP_MOD_DELETE:
-                       case LDAP_MOD_REPLACE: {
-                               BerVarray vals = NULL, nvals = NULL;
-                               Attribute *a;
-                               delrec *d = NULL;
-
-                               a = attr_find( e->e_attrs, ml->sml_desc );
-
-                               if ( ml->sml_op == LDAP_MOD_REPLACE ) {
-                                       vals = ml->sml_values;
-                                       nvals = ml->sml_nvalues;
-                                       ml->sml_values = NULL;
-                                       ml->sml_nvalues = NULL;
-                               }
+       /* Basic syntax checks are OK. Do the actual settings. */
+       for ( ml = op->orm_modlist; ml; ml = ml->sml_next ) {
+               ct = config_find_table( colst, nocs, ml->sml_desc );
+               if ( !ct ) continue;
 
-                               if ( ml->sml_values )
-                                       d = dels;
+               s = attr_find( save_attrs, ml->sml_desc );
+               a = attr_find( e->e_attrs, ml->sml_desc );
 
-                               /* If we didn't delete the whole attribute */
-                               if ( ml->sml_values && a ) {
-                                       struct berval *mvals;
-                                       int j;
+               switch (ml->sml_op) {
+               case LDAP_MOD_DELETE:
+               case LDAP_MOD_REPLACE: {
+                       BerVarray vals = NULL, nvals = NULL;
+                       delrec *d = NULL;
 
-                                       if ( ml->sml_nvalues )
-                                               mvals = ml->sml_nvalues;
-                                       else
-                                               mvals = ml->sml_values;
-
-                                       /* use the indexes we saved up above */
-                                       for (i=0; i < d->nidx; i++) {
-                                               struct berval bv = *mvals++;
-                                               if ( a->a_desc->ad_type->sat_flags & SLAP_AT_ORDERED &&
-                                                       bv.bv_val[0] == '{' ) {
-                                                       ptr = strchr( bv.bv_val, '}' ) + 1;
-                                                       bv.bv_len -= ptr - bv.bv_val;
-                                                       bv.bv_val = ptr;
-                                               }
-                                               ca->line = bv.bv_val;
-                                               ca->valx = d->idx[i];
-                                               rc = config_del_vals( ct, ca );
-                                               if ( rc != LDAP_SUCCESS ) break;
-                                               for (j=i+1; j < d->nidx; j++)
-                                                       if ( d->idx[j] >d->idx[i] )
-                                                               d->idx[j]--;
+                       if ( ml->sml_op == LDAP_MOD_REPLACE ) {
+                               vals = ml->sml_values;
+                               nvals = ml->sml_nvalues;
+                               ml->sml_values = NULL;
+                               ml->sml_nvalues = NULL;
+                       }
+
+                       if ( ml->sml_values )
+                               d = dels;
+
+                       /* If we didn't delete the whole attribute */
+                       if ( ml->sml_values && a ) {
+                               struct berval *mvals;
+                               int j;
+
+                               if ( ml->sml_nvalues )
+                                       mvals = ml->sml_nvalues;
+                               else
+                                       mvals = ml->sml_values;
+
+                               /* use the indexes we saved up above */
+                               for (i=0; i < d->nidx; i++) {
+                                       struct berval bv = *mvals++;
+                                       if ( a->a_desc->ad_type->sat_flags & SLAP_AT_ORDERED &&
+                                               bv.bv_val[0] == '{' ) {
+                                               ptr = strchr( bv.bv_val, '}' ) + 1;
+                                               bv.bv_len -= ptr - bv.bv_val;
+                                               bv.bv_val = ptr;
                                        }
-                               } else {
-                                       ca->valx = -1;
-                                       ca->line = NULL;
+                                       ca->line = bv.bv_val;
+                                       ca->valx = d->idx[i];
                                        rc = config_del_vals( ct, ca );
-                                       if ( rc ) rc = LDAP_OTHER;
+                                       if ( rc != LDAP_SUCCESS ) break;
+                                       if ( s )
+                                               s->a_flags |= SLAP_ATTR_IXDEL;
+                                       for (j=i+1; j < d->nidx; j++)
+                                               if ( d->idx[j] >d->idx[i] )
+                                                       d->idx[j]--;
                                }
-                               if ( ml->sml_values ) {
-                                       d = d->next;
-                                       ch_free( dels );
-                                       dels = d;
-                               }
-                               if ( ml->sml_op == LDAP_MOD_REPLACE ) {
-                                       ml->sml_values = vals;
-                                       ml->sml_nvalues = nvals;
-                               }
-                               if ( !vals || rc != LDAP_SUCCESS )
-                                       break;
-                               }
-                               /* FALLTHRU: LDAP_MOD_REPLACE && vals */
+                       } else {
+                               ca->valx = -1;
+                               ca->line = NULL;
+                               rc = config_del_vals( ct, ca );
+                               if ( rc ) rc = LDAP_OTHER;
+                               if ( s )
+                                       s->a_flags |= SLAP_ATTR_IXDEL;
+                       }
+                       if ( ml->sml_values ) {
+                               d = d->next;
+                               ch_free( dels );
+                               dels = d;
+                       }
+                       if ( ml->sml_op == LDAP_MOD_REPLACE ) {
+                               ml->sml_values = vals;
+                               ml->sml_nvalues = nvals;
+                       }
+                       if ( !vals || rc != LDAP_SUCCESS )
+                               break;
+                       }
+                       /* FALLTHRU: LDAP_MOD_REPLACE && vals */
+
+               case LDAP_MOD_ADD:
+                       for (i=0; ml->sml_values[i].bv_val; i++) {
+                               ca->line = ml->sml_values[i].bv_val;
+                               ca->valx = -1;
+                               rc = config_modify_add( ct, ca, ml->sml_desc, i );
+                               if ( rc )
+                                       goto out;
+                               a->a_flags |= SLAP_ATTR_IXADD;
+                       }
+                       break;
+               }
+       }
 
-                       case LDAP_MOD_ADD:
-                               for (i=0; ml->sml_values[i].bv_val; i++) {
-                                       ca->line = ml->sml_values[i].bv_val;
+out:
+       /* Undo for a failed operation */
+       if ( rc != LDAP_SUCCESS ) {
+               for ( s = save_attrs; s; s = s->a_next ) {
+                       if ( s->a_flags & SLAP_ATTR_IXDEL ) {
+                               s->a_flags &= ~(SLAP_ATTR_IXDEL|SLAP_ATTR_IXADD);
+                               ct = config_find_table( colst, nocs, s->a_desc );
+                               a = attr_find( e->e_attrs, s->a_desc );
+                               if ( a ) {
+                                       /* clear the flag so the add check below will skip it */
+                                       a->a_flags &= ~(SLAP_ATTR_IXDEL|SLAP_ATTR_IXADD);
                                        ca->valx = -1;
-                                       if ( ml->sml_desc->ad_type->sat_flags & SLAP_AT_ORDERED &&
-                                               ca->line[0] == '{' )
-                                       {
-                                               ptr = strchr( ca->line + 1, '}' );
-                                               if ( ptr ) {
-                                                       char    *next;
-
-                                                       ca->valx = strtol( ca->line + 1, &next, 0 );
-                                                       if ( next == ca->line + 1 || next[ 0 ] != '}' ) {
-                                                               rc = LDAP_OTHER;
-                                                               goto out;
-                                                       }
-                                                       ca->line = ptr+1;
-                                               }
-                                       }
-                                       rc = config_parse_add( ct, ca, i );
-                                       if ( rc ) {
-                                               rc = LDAP_OTHER;
-                                               goto out;
+                                       ca->line = NULL;
+                                       config_del_vals( ct, ca );
+                               }
+                               for ( i=0; !BER_BVISNULL( &s->a_vals[i] ); i++ ) {
+                                       ca->line = s->a_vals[i].bv_val;
+                                       ca->valx = -1;
+                                       config_modify_add( ct, ca, s->a_desc, i );
+                               }
+                       }
+               }
+               for ( a = e->e_attrs; a; a = a->a_next ) {
+                       if ( a->a_flags & SLAP_ATTR_IXADD ) {
+                               ct = config_find_table( colst, nocs, a->a_desc );
+                               ca->valx = -1;
+                               ca->line = NULL;
+                               config_del_vals( ct, ca );
+                               s = attr_find( save_attrs, a->a_desc );
+                               if ( s ) {
+                                       s->a_flags &= ~(SLAP_ATTR_IXDEL|SLAP_ATTR_IXADD);
+                                       for ( i=0; !BER_BVISNULL( &s->a_vals[i] ); i++ ) {
+                                               ca->line = s->a_vals[i].bv_val;
+                                               ca->valx = -1;
+                                               config_modify_add( ct, ca, s->a_desc, i );
                                        }
                                }
-
-                               break;
                        }
                }
        }
 
-out:
        if ( ca->cleanup )
                ca->cleanup( ca );
+out_noop:
        if ( rc == LDAP_SUCCESS ) {
                attrs_free( save_attrs );
        } else {
@@ -5215,9 +5329,9 @@ config_build_entry( Operation *op, SlapReply *rs, CfEntryInfo *parent,
        ce->ce_parent = parent;
        if ( parent ) {
                pdn = parent->ce_entry->e_nname;
-               if ( parent->ce_kids && parent->ce_kids->ce_type < ce->ce_type )
+               if ( parent->ce_kids )
                        for ( ceprev = parent->ce_kids; ceprev->ce_sibs &&
-                               ceprev->ce_type < ce->ce_type;
+                               ceprev->ce_type <= ce->ce_type;
                                ceprev = ceprev->ce_sibs );
        } else {
                BER_BVZERO( &pdn );