X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=servers%2Fslapd%2Fcompare.c;h=fdfccea5193fe8ad345aee050ae58846b595ac8a;hb=e4b899df95374d56f79350a146fea649db3b2a57;hp=2bffeafbde6fbdcab0c0c31239cbec94b0fde696;hpb=2fdbc553746c44df7ec15c88b73d608743bfa030;p=openldap diff --git a/servers/slapd/compare.c b/servers/slapd/compare.c index 2bffeafbde..fdfccea519 100644 --- a/servers/slapd/compare.c +++ b/servers/slapd/compare.c @@ -1,10 +1,18 @@ /* $OpenLDAP$ */ -/* - * Copyright 1998-2003 The OpenLDAP Foundation, All Rights Reserved. - * COPYING RESTRICTIONS APPLY, see COPYRIGHT file +/* This work is part of OpenLDAP Software . + * + * Copyright 1998-2003 The OpenLDAP Foundation. + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted only as authorized by the OpenLDAP + * Public License. + * + * A copy of this license is available in the file LICENSE in the + * top-level directory of the distribution or, alternatively, at + * . */ -/* - * Copyright (c) 1995 Regents of the University of Michigan. +/* Portions Copyright (c) 1995 Regents of the University of Michigan. * All rights reserved. * * Redistribution and use in source and binary forms are permitted @@ -19,6 +27,7 @@ #include #include +#include #include "ldap_pvt.h" #include "slap.h" @@ -38,18 +47,12 @@ do_compare( ) { Entry *entry = NULL; - Entry *fentry = NULL; struct berval dn = { 0, NULL }; struct berval desc = { 0, NULL }; struct berval value = { 0, NULL }; AttributeAssertion ava = { NULL, { 0, NULL } }; - Backend *be; int manageDSAit; -#ifdef LDAP_SLAPI - Slapi_PBlock *pb = op->o_pb; -#endif - ava.aa_desc = NULL; #ifdef NEW_LOGGING @@ -112,7 +115,7 @@ do_compare( goto cleanup; } - rs->sr_err = dnPrettyNormal( NULL, &dn, &op->o_req_dn, &op->o_req_ndn ); + rs->sr_err = dnPrettyNormal( NULL, &dn, &op->o_req_dn, &op->o_req_ndn, op->o_tmpmemctx ); if( rs->sr_err != LDAP_SUCCESS ) { #ifdef NEW_LOGGING LDAP_LOG( OPERATION, INFO, @@ -132,15 +135,10 @@ do_compare( goto cleanup; } -#ifdef SLAP_NVALUES rs->sr_err = asserted_value_validate_normalize( ava.aa_desc, ava.aa_desc->ad_type->sat_equality, SLAP_MR_EQUALITY|SLAP_MR_VALUE_OF_ASSERTION_SYNTAX, - &value, &ava.aa_value, &rs->sr_text ); -#else - rs->sr_err = value_validate_normalize( ava.aa_desc, SLAP_MR_EQUALITY, - &value, &ava.aa_value, &rs->sr_text ); -#endif + &value, &ava.aa_value, &rs->sr_text, op->o_tmpmemctx ); if( rs->sr_err != LDAP_SUCCESS ) { send_ldap_result( op, rs ); goto cleanup; @@ -172,8 +170,6 @@ do_compare( goto cleanup; } - fentry = entry; - } else if ( bvmatch( &op->o_req_ndn, &global_schemandn ) ) { #ifdef NEW_LOGGING LDAP_LOG( OPERATION, ARGS, @@ -202,12 +198,11 @@ do_compare( rs->sr_err = 0; goto cleanup; } - fentry = entry; } if( entry ) { rs->sr_err = compare_entry( op, entry, &ava ); - if( fentry) entry_free( fentry ); + entry_free( entry ); send_ldap_result( op, rs ); @@ -263,33 +258,38 @@ do_compare( ava.aa_desc->ad_cname.bv_val, 0 ); #if defined( LDAP_SLAPI ) - slapi_x_pblock_set_operation( pb, op ); - slapi_pblock_set( pb, SLAPI_COMPARE_TARGET, (void *)dn.bv_val ); - slapi_pblock_set( pb, SLAPI_MANAGEDSAIT, (void *)manageDSAit ); - slapi_pblock_set( pb, SLAPI_COMPARE_TYPE, (void *)desc.bv_val ); - slapi_pblock_set( pb, SLAPI_COMPARE_VALUE, (void *)&value ); - - rs->sr_err = doPluginFNs( op->o_bd, SLAPI_PLUGIN_PRE_COMPARE_FN, pb ); - if ( rs->sr_err != 0 ) { - /* - * A preoperation plugin failure will abort the - * entire operation. - */ +#define pb op->o_pb + if ( pb ) { + slapi_int_pblock_set_operation( pb, op ); + slapi_pblock_set( pb, SLAPI_COMPARE_TARGET, (void *)dn.bv_val ); + slapi_pblock_set( pb, SLAPI_MANAGEDSAIT, (void *)manageDSAit ); + slapi_pblock_set( pb, SLAPI_COMPARE_TYPE, (void *)desc.bv_val ); + slapi_pblock_set( pb, SLAPI_COMPARE_VALUE, (void *)&value ); + + rs->sr_err = doPluginFNs( op->o_bd, SLAPI_PLUGIN_PRE_COMPARE_FN, pb ); + if ( rs->sr_err < 0 ) { + /* + * A preoperation plugin failure will abort the + * entire operation. + */ #ifdef NEW_LOGGING - LDAP_LOG( OPERATION, INFO, "do_compare: compare preoperation plugin " - "failed\n", 0, 0, 0); + LDAP_LOG( OPERATION, INFO, "do_compare: compare preoperation plugin " + "failed\n", 0, 0, 0); #else - Debug(LDAP_DEBUG_TRACE, "do_compare: compare preoperation plugin " - "failed.\n", 0, 0, 0); + Debug(LDAP_DEBUG_TRACE, "do_compare: compare preoperation plugin " + "failed.\n", 0, 0, 0); #endif - if ( slapi_pblock_get( pb, SLAPI_RESULT_CODE, (void *)&rs->sr_err ) != 0) - rs->sr_err = LDAP_OTHER; - goto cleanup; + if ( ( slapi_pblock_get( op->o_pb, SLAPI_RESULT_CODE, (void *)&rs->sr_err ) != 0 ) || + rs->sr_err == LDAP_SUCCESS ) { + rs->sr_err = LDAP_OTHER; + } + goto cleanup; + } } #endif /* defined( LDAP_SLAPI ) */ if ( op->o_bd->be_compare ) { - op->oq_compare.rs_ava = &ava; + op->orc_ava = &ava; op->o_bd->be_compare( op, rs ); } else { send_ldap_error( op, rs, LDAP_UNWILLING_TO_PERFORM, @@ -297,7 +297,7 @@ do_compare( } #if defined( LDAP_SLAPI ) - if ( doPluginFNs( op->o_bd, SLAPI_PLUGIN_POST_COMPARE_FN, pb ) != 0 ) { + if ( pb && doPluginFNs( op->o_bd, SLAPI_PLUGIN_POST_COMPARE_FN, pb ) < 0 ) { #ifdef NEW_LOGGING LDAP_LOG( OPERATION, INFO, "do_compare: compare postoperation plugins " "failed\n", 0, 0, 0 ); @@ -309,9 +309,9 @@ do_compare( #endif /* defined( LDAP_SLAPI ) */ cleanup: - free( op->o_req_dn.bv_val ); - free( op->o_req_ndn.bv_val ); - if ( ava.aa_value.bv_val ) free( ava.aa_value.bv_val ); + op->o_tmpfree( op->o_req_dn.bv_val, op->o_tmpmemctx ); + op->o_tmpfree( op->o_req_ndn.bv_val, op->o_tmpmemctx ); + if ( ava.aa_value.bv_val ) op->o_tmpfree( ava.aa_value.bv_val, op->o_tmpmemctx ); return rs->sr_err; } @@ -336,15 +336,11 @@ static int compare_entry( { rc = LDAP_COMPARE_FALSE; -#ifdef SLAP_NVALUES if ( value_find_ex( ava->aa_desc, SLAP_MR_ATTRIBUTE_VALUE_NORMALIZED_MATCH | SLAP_MR_ASSERTED_VALUE_NORMALIZED_MATCH, a->a_nvals, - &ava->aa_value ) == 0 ) -#else - if ( value_find( ava->aa_desc, a->a_vals, &ava->aa_value ) == 0 ) -#endif + &ava->aa_value, op->o_tmpmemctx ) == 0 ) { rc = LDAP_COMPARE_TRUE; break;