]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/mods.c
NVALUES: fix a couple of value_find_ex() calls
[openldap] / servers / slapd / mods.c
index 0fed9643b45f1a6e997ed71b22d5471f9b7f3ff3..73aac72e39a97d8280c08299878b2ff76f8c0149 100644 (file)
@@ -30,6 +30,9 @@ modify_check_duplicates(
 {
        int             i, j, numvals = 0, nummods,
                        rc = LDAP_SUCCESS, matched;
+#ifdef SLAP_NVALUES
+       /* needs major reworking */
+#else
        BerVarray       nvals = NULL, nmods = NULL;
 
        /*
@@ -121,23 +124,24 @@ modify_check_duplicates(
                goto return_results;
        }
 
-       for ( i = 0; mods[ i ].bv_val != NULL; i++ ) {
+       for ( i=0; mods[i].bv_val != NULL; i++ ) {
                rc = value_normalize( ad, SLAP_MR_EQUALITY,
-                       &mods[ i ], &nmods[ i ], text );
+                       &mods[i], &nmods[i], text );
 
                if ( rc != LDAP_SUCCESS ) {
-                       nmods[ i ].bv_val = NULL;
+                       nmods[i].bv_val = NULL;
                        goto return_results;
                }
 
                if ( numvals > 0 && numvals < nummods ) {
-                       for ( matched = 0, j = 0; nvals[ j ].bv_val; j++ ) {
+                       for ( matched=0, j=0; nvals[j].bv_val; j++ ) {
                                int match;
 
                                rc = (*mr->smr_match)( &match,
                                        SLAP_MR_ATTRIBUTE_SYNTAX_MATCH,
                                        ad->ad_type->sat_syntax,
                                        mr, &nmods[ i ], &nvals[ j ] );
+
                                if ( rc != LDAP_SUCCESS ) {
                                        nmods[ i + 1 ].bv_val = NULL;
                                        *text = textbuf;
@@ -270,6 +274,7 @@ return_results:;
                ber_bvarray_free( nmods );
        }
 
+#endif
        return rc;
 }
 
@@ -310,6 +315,11 @@ modify_add_values(
 
        /* check if the values we're adding already exist */
        if( mr == NULL || !mr->smr_match ) {
+#ifdef SLAP_NVALUES
+               /* we should have no normalized values as there is no equality rule */
+               /* assert( mod->sm_nvalues[0].bv_val == NULL); */
+#endif
+
                if ( a != NULL ) {
                        /* do not allow add of additional attribute
                                if no equality rule exists */
@@ -323,9 +333,13 @@ modify_add_values(
                for ( i = 0; mod->sm_bvalues[i].bv_val != NULL; i++ ) {
                        /* test asserted values against existing values */
                        if( a ) {
+#ifdef SLAP_NVALUES
+                               /* we should have no normalized values as there
+                                       is no equality rule */
+                               assert( a->a_nvals == NULL);
+#endif
                                for( matched = 0, j = 0; a->a_vals[j].bv_val != NULL; j++ ) {
-                                       if ( bvmatch( &mod->sm_bvalues[i],
-                                               &a->a_vals[j] ) ) {
+                                       if ( bvmatch( &mod->sm_bvalues[i], &a->a_vals[j] ) ) {
                                                if ( permissive ) {
                                                        matched++;
                                                        continue;
@@ -360,6 +374,16 @@ modify_add_values(
                }
 
        } else {
+#ifdef SLAP_NVALUES
+               /* no normalization is done in this routine nor
+                * in the matching routines called by this routine. 
+                * values are now normalized once on input to the
+                * server (whether from LDAP or from the underlying
+                * database).
+                * This should outperform the old code.  No numbers
+                * are available yet.
+                */
+#else
                /*
                 * The original code performs ( n ) normalizations 
                 * and ( n * ( n - 1 ) / 2 ) matches, which hide
@@ -386,34 +410,61 @@ modify_add_values(
                 * Maybe we could switch to the new algorithm when
                 * the number of values overcomes a given threshold?
                 */
+#endif
 
                int             rc;
 
-               if ( mod->sm_bvalues[ 1 ].bv_val == 0 ) {
+               if ( mod->sm_bvalues[1].bv_val == 0 ) {
                        if ( a != NULL ) {
-                               struct berval   asserted;
                                int             i;
-
+#ifndef SLAP_NVALUES
+                               struct berval   asserted;
                                rc = value_normalize( mod->sm_desc, SLAP_MR_EQUALITY,
                                        &mod->sm_bvalues[ 0 ], &asserted, text );
-
                                if ( rc != LDAP_SUCCESS ) {
                                        return rc;
                                }
+#endif
 
                                for ( matched = 0, i = 0; a->a_vals[ i ].bv_val; i++ ) {
                                        int     match;
 
+#ifdef SLAP_NVALUES
+                                       if( mod->sm_nvalues ) {
+                                               rc = value_match( &match, mod->sm_desc, mr,
+                                                       SLAP_MR_EQUALITY
+                                                               | SLAP_MR_VALUE_OF_ASSERTION_SYNTAX
+                                                               | SLAP_MR_ASSERTED_VALUE_NORMALIZED_MATCH
+                                                               | SLAP_MR_ATTRIBUTE_VALUE_NORMALIZED_MATCH,
+                                                       &a->a_nvals[i],
+                                                       &mod->sm_nvalues[0],
+                                                       text );
+
+                                       } else {
+                                               rc = value_match( &match, mod->sm_desc, mr,
+                                                       SLAP_MR_EQUALITY
+                                                               | SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
+                                                       &a->a_vals[i],
+                                                       &mod->sm_values[0],
+                                                       text );
+                                       }
+
+#else
                                        rc = value_match( &match, mod->sm_desc, mr,
                                                SLAP_MR_ATTRIBUTE_SYNTAX_MATCH,
-                                               &a->a_vals[ i ], &asserted, text );
+                                               &a->a_vals[i],
+                                               &asserted,
+                                               text );
+#endif
 
                                        if( rc == LDAP_SUCCESS && match == 0 ) {
                                                if ( permissive ) {
                                                        matched++;
                                                        continue;
                                                }
+#ifndef SLAP_NVALUES
                                                free( asserted.bv_val );
+#endif
                                                *text = textbuf;
                                                snprintf( textbuf, textlen,
                                                        "modify/%s: %s: value #0 already exists",
@@ -421,6 +472,9 @@ modify_add_values(
                                                return LDAP_TYPE_OR_VALUE_EXISTS;
                                        }
                                }
+#ifndef SLAP_NVALUES
+                               free( asserted.bv_val );
+#endif
                                if ( permissive && matched == i ) {
                                        /* values already exist; do nothing */
                                        return LDAP_SUCCESS;
@@ -473,7 +527,9 @@ modify_delete_values(
        int             i, j, k, rc = LDAP_SUCCESS;
        Attribute       *a;
        MatchingRule    *mr = mod->sm_desc->ad_type->sat_equality;
+#ifndef SLAP_NVALUES
        BerVarray       nvals = NULL;
+#endif
        char            dummy = '\0';
 
        /*
@@ -519,6 +575,7 @@ modify_delete_values(
                return LDAP_NO_SUCH_ATTRIBUTE;
        }
 
+#ifndef SLAP_NVALUES
        /* find each value to delete */
        for ( j = 0; a->a_vals[ j ].bv_val != NULL; j++ )
                /* count existing values */ ;
@@ -545,10 +602,12 @@ modify_delete_values(
                        goto return_results;
                }
        }
+#endif
 
-       for ( i = 0; mod->sm_bvalues[ i ].bv_val != NULL; i++ ) {
-               struct  berval asserted;
+       for ( i = 0; mod->sm_values[i].bv_val != NULL; i++ ) {
                int     found = 0;
+#ifndef SLAP_NVALUES
+               struct  berval asserted;
 
                /* normalize the value to be deleted */
                rc = value_normalize( mod->sm_desc, SLAP_MR_EQUALITY,
@@ -559,20 +618,49 @@ modify_delete_values(
                }
 
                /* search it */
-               for ( j = 0; nvals[ j ].bv_val != NULL; j++ ) {
+               for ( j = 0; nvals[ j ].bv_val != NULL; j++ )
+#else
+               for ( j = 0; a->a_vals[j].bv_val != NULL; j++ )
+#endif
+               {
                        int match;
 
-                       if ( nvals[ j ].bv_val == &dummy ) {
+#ifndef SLAP_NVALUES
+                       if ( nvals[j].bv_val == &dummy ) {
                                continue;
                        }
 
+#endif
+#ifdef SLAP_NVALUES
+                       if( mod->sm_nvalues ) {
+                               assert( a->a_nvals );
+                               rc = (*mr->smr_match)( &match,
+                                       SLAP_MR_VALUE_OF_ASSERTION_SYNTAX
+                                               | SLAP_MR_ASSERTED_VALUE_NORMALIZED_MATCH
+                                               | SLAP_MR_ATTRIBUTE_VALUE_NORMALIZED_MATCH,
+                                       a->a_desc->ad_type->sat_syntax,
+                                       mr, &a->a_nvals[j],
+                                       &mod->sm_nvalues[i] );
+                       } else {
+                               assert( a->a_nvals == NULL );
+                               rc = (*mr->smr_match)( &match,
+                                       SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
+                                       a->a_desc->ad_type->sat_syntax,
+                                       mr, &a->a_nvals[j],
+                                       &mod->sm_values[i] );
+                       }
+#else
                        rc = (*mr->smr_match)( &match,
                                SLAP_MR_ATTRIBUTE_SYNTAX_MATCH,
                                a->a_desc->ad_type->sat_syntax,
-                               mr, &nvals[ j ], &asserted );
+                               mr, &nvals[ j ],
+                               &asserted );
+#endif
 
                        if ( rc != LDAP_SUCCESS ) {
+#ifndef SLAP_NVALUES
                                free( asserted.bv_val );
+#endif
                                *text = textbuf;
                                snprintf( textbuf, textlen,
                                        "%s: matching rule failed",
@@ -587,13 +675,24 @@ modify_delete_values(
                        found = 1;
 
                        /* delete value and mark it as dummy */
+#ifdef SLAP_NVALUES
+                       free( a->a_vals[j].bv_val );
+                       a->a_vals[j].bv_val = &dummy;
+                       if( a->a_nvals ) {
+                               free( a->a_nvals[j].bv_val );
+                               a->a_nvals[j].bv_val = &dummy;
+                       }
+#else
                        free( nvals[ j ].bv_val );
                        nvals[ j ].bv_val = &dummy;
+#endif
 
                        break;
                }
 
+#ifndef SLAP_NVALUES
                free( asserted.bv_val );
+#endif
 
                if ( found == 0 ) {
                        *text = textbuf;
@@ -606,33 +705,49 @@ modify_delete_values(
        }
 
        /* compact array skipping dummies */
-       for ( k = 0, j = 0; nvals[ k ].bv_val != NULL; j++, k++ ) {
-
+#ifdef SLAP_NVALUES
+       for ( k = 0, j = 0; a->a_vals[k].bv_val != NULL; k++ )
+#else
+       for ( k = 0, j = 0; nvals[k].bv_val != NULL; j++, k++ )
+#endif
+       {
+#ifdef SLAP_NVALUES
+               /* skip dummies */
+               if( a->a_vals[k].bv_val == &dummy ) {
+                       assert( a->a_nvals == NULL || a->a_nvals[k].bv_val == &dummy );
+                       continue;
+               }
+#else
                /* delete and skip dummies */ ;
                for ( ; nvals[ k ].bv_val == &dummy; k++ ) {
                        free( a->a_vals[ k ].bv_val );
                }
-
+#endif
                if ( j != k ) {
                        a->a_vals[ j ] = a->a_vals[ k ];
 #ifdef SLAP_NVALUES
                        if (a->a_nvals) {
-                               free( a->a_nvals[j].bv_val );
                                a->a_nvals[ j ] = a->a_nvals[ k ];
                        }
 #endif
                }
 
+#ifndef SLAP_NVALUES
                if ( a->a_vals[ k ].bv_val == NULL ) {
                        break;
                }
+#else
+               j++;
+#endif
        }
-       a->a_vals[ j ].bv_val = NULL;
+
+       a->a_vals[j].bv_val = NULL;
 #ifdef SLAP_NVALUES
-       a->a_nvals[ j ].bv_val = NULL;
-#endif
+       if (a->a_nvals) a->a_nvals[j].bv_val = NULL;
+#else
 
        assert( i == k - j );
+#endif
 
        /* if no values remain, delete the entire attribute */
        if ( a->a_vals[0].bv_val == NULL ) {
@@ -646,6 +761,7 @@ modify_delete_values(
        }
 
 return_results:;
+#ifndef SLAP_NVALUES
        if ( nvals ) {
                /* delete the remaining normalized values */
                for ( j = 0; nvals[ j ].bv_val != NULL; j++ ) {
@@ -655,6 +771,7 @@ return_results:;
                }
                ber_memfree( nvals );
        }
+#endif
 
        return rc;
 }