]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-sql/modify.c
fix access checking; fix operational attrs addition
[openldap] / servers / slapd / back-sql / modify.c
index de3d63aa92e4c8d4a7592fe3fa4fda877af4ae3a..c9060f98eb4a62bb3cc81526517627dbeb8e7a34 100644 (file)
-/*
- *      Copyright 1999, Dmitry Kovalev <mit@openldap.org>, All rights reserved.
+/* $OpenLDAP$ */
+/* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- *      Redistribution and use in source and binary forms are permitted only
- *      as authorized by the OpenLDAP Public License.  A copy of this
- *      license is available at http://www.OpenLDAP.org/license.html or
- *      in file LICENSE in the top-level directory of the distribution.
+ * Copyright 1999-2005 The OpenLDAP Foundation.
+ * Portions Copyright 1999 Dmitry Kovalev.
+ * Portions Copyright 2002 Pierangelo Masarati.
+ * 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
+ * <http://www.OpenLDAP.org/license.html>.
+ */
+/* ACKNOWLEDGEMENTS:
+ * This work was initially developed by Dmitry Kovalev for inclusion
+ * by OpenLDAP Software.  Additional significant contributors include
+ * Pierangelo Masarati.
  */
 
 #include "portable.h"
 
-#ifdef SLAPD_SQL
-
 #include <stdio.h>
 #include <sys/types.h>
 #include "ac/string.h"
-#include "slap.h"
-#include "ldap_pvt.h"
-#include "back-sql.h"
-#include "sql-wrap.h"
-#include "schema-map.h"
-#include "entry-id.h"
-#include "util.h"
-
-/*
- * PostgreSQL 7.0 doesn't work without :(
- */
-#define        BACKSQL_REALLOC_STMT
-
-/*
- * Skip:
- * - the first occurrence of objectClass, which is used
- *   to determine how to build the SQL entry (FIXME ?!?)
- * - operational attributes
- *   empty attributes (FIXME ?!?)
- */
-#define        backsql_attr_skip(ad,vals) \
-       ( \
-               ( (ad) == slap_schema.si_ad_objectClass \
-                               && (vals)[ 1 ].bv_val == NULL ) \
-               || is_at_operational( (ad)->ad_type ) \
-               || ( (vals)[ 0 ].bv_val == NULL ) \
-       )
-
-static int
-backsql_modify_internal(
-       Operation               *op,
-       SlapReply               *rs,
-       SQLHDBC                 dbh, 
-       backsql_oc_map_rec      *oc,
-       backsql_entryID         *e_id,
-       Modifications           *modlist )
-{
-       backsql_info    *bi = (backsql_info*)op->o_bd->be_private;
-       RETCODE         rc;
-       SQLHSTMT        sth;
-       Modifications   *ml;
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-               "traversing modifications list\n", 0, 0, 0 );
 
-#ifndef BACKSQL_REALLOC_STMT
-       SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-
-       for ( ml = modlist; ml != NULL; ml = ml->sml_next ) {
-               AttributeDescription    *ad;
-               backsql_at_map_rec      *at = NULL;
-               struct berval           *at_val;
-               Modification            *c_mod;
-               int                     i;
-               /* first parameter no, parameter order */
-               SQLUSMALLINT            pno, po;
-               /* procedure return code */
-               int                     prc;
-
-#ifdef BACKSQL_REALLOC_STMT
-               SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-
-               c_mod = &ml->sml_mod;
-               ad = c_mod->sm_desc;
-
-               Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-                       "modifying attribute '%s'\n",
-                       ad->ad_cname.bv_val, 0, 0 );
-
-               if ( backsql_attr_skip( ad, c_mod->sm_bvalues ) ) {
-                       continue;
-               }
-
-               at = backsql_ad2at( oc, ad );
-               if ( at == NULL ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-                               "attribute provided is not registered "
-                               "in objectClass '%s'\n",
-                               ad->ad_cname.bv_val, 0, 0 );
-
-                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                               rs->sr_text = "operation not permitted "
-                                       "within namingContext";
-                               goto done;
-                       }
-
-                       continue;
-               }
-  
-               switch( c_mod->sm_op ) {
-               case LDAP_MOD_REPLACE: {
-                       SQLHSTMT asth;
-                       BACKSQL_ROW_NTS row;
-                       
-                       Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-                               "replacing values for attribute '%s'\n",
-                               at->ad->ad_cname.bv_val, 0, 0 );
-
-                       if ( at->add_proc == NULL ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "add procedure is not defined "
-                                       "for attribute '%s' "
-                                       "- unable to perform replacements\n",
-                                       at->ad->ad_cname.bv_val, 0, 0 );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                                       rs->sr_text = "operation not permitted "
-                                               "within namingContext";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-
-                       if ( at->delete_proc == NULL ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "delete procedure is not defined "
-                                       "for attribute '%s' "
-                                       "- adding only\n",
-                                       at->ad->ad_cname.bv_val, 0, 0 );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                                       rs->sr_text = "operation not permitted "
-                                               "within namingContext";
-                                       goto done;
-                               }
-
-                               goto add_only;
-                       }
-                       
-del_all:
-                       rc = backsql_Prepare( dbh, &asth, at->query, 0 );
-                       if ( rc != SQL_SUCCESS ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "error preparing query\n", 0, 0, 0 );
-                               backsql_PrintErrors( bi->db_env, dbh, 
-                                               asth, rc );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_OTHER;
-                                       rs->sr_text = "SQL-backend error";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-
-                       rc = backsql_BindParamID( asth, 1, &e_id->keyval );
-                       if ( rc != SQL_SUCCESS ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "error binding key value parameter\n",
-                                       0, 0, 0 );
-                               backsql_PrintErrors( bi->db_env, dbh, 
-                                               asth, rc );
-                               SQLFreeStmt( asth, SQL_DROP );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_OTHER;
-                                       rs->sr_text = "SQL-backend error";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-                       
-                       rc = SQLExecute( asth );
-                       if ( !BACKSQL_SUCCESS( rc ) ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "error executing attribute query\n",
-                                       0, 0, 0 );
-                               backsql_PrintErrors( bi->db_env, dbh, 
-                                               asth, rc );
-                               SQLFreeStmt( asth, SQL_DROP );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_OTHER;
-                                       rs->sr_text = "SQL-backend error";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-
-                       backsql_BindRowAsStrings( asth, &row );
-                       rc = SQLFetch( asth );
-                       for ( ; BACKSQL_SUCCESS( rc ); rc = SQLFetch( asth ) ) {
-                               for ( i = 0; i < row.ncols; i++ ) {
-                                       if ( BACKSQL_IS_DEL( at->expect_return ) ) {
-                                               pno = 1;
-                                               SQLBindParameter(sth, 1,
-                                                       SQL_PARAM_OUTPUT,
-                                                       SQL_C_ULONG,
-                                                       SQL_INTEGER,
-                                                       0, 0, &prc, 0, 0 );
-                                       } else {
-                                               pno = 0;
-                                       }
-                                       po = ( BACKSQL_IS_DEL( at->param_order ) ) > 0;
-                                       SQLBindParameter( sth, pno + 1 + po,
-                                               SQL_PARAM_INPUT,
-                                               SQL_C_ULONG, SQL_INTEGER,
-                                               0, 0, &e_id->keyval, 0, 0 );
-
-                                       /*
-                                        * check for syntax needed here 
-                                        * maybe need binary bind?
-                                        */
-                                       SQLBindParameter(sth, pno + 2 - po,
-                                               SQL_PARAM_INPUT,
-                                               SQL_C_CHAR, SQL_CHAR,
-                                               0, 0, row.cols[ i ],
-                                               strlen( row.cols[ i ] ), 0 );
-                        
-                                       Debug( LDAP_DEBUG_TRACE, 
-                                               "backsql_modify_internal(): "
-                                               "executing '%s'\n",
-                                               at->delete_proc, 0, 0 );
-                                       rc = SQLExecDirect( sth,
-                                               at->delete_proc, SQL_NTS );
-                                       if ( rc != SQL_SUCCESS ) {
-                                               Debug( LDAP_DEBUG_TRACE,
-                                                       "backsql_modify_internal(): "
-                                                       "delete_proc "
-                                                       "execution failed\n",
-                                                       0, 0, 0 );
-                                               backsql_PrintErrors( bi->db_env,
-                                                               dbh, sth, rc );
-
-                                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                                       rs->sr_err = LDAP_OTHER;
-                                                       rs->sr_text = "SQL-backend error";
-                                                       goto done;
-                                               }
-                                       }
-#ifdef BACKSQL_REALLOC_STMT
-                                       SQLFreeStmt( sth, SQL_DROP );
-                                       SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-                               }
-                       }
-                       backsql_FreeRow( &row );
-                       SQLFreeStmt( asth, SQL_DROP );
-               }
-                                      
-               /*
-                * PASSTHROUGH - to add new attributes -- do NOT add break
-                */
-               case LDAP_MOD_ADD:
-               case SLAP_MOD_SOFTADD:
-add_only:;
-                       if ( at->add_proc == NULL ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "add procedure is not defined "
-                                       "for attribute '%s'\n",
-                                       at->ad->ad_cname.bv_val, 0, 0 );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                                       rs->sr_text = "operation not permitted "
-                                               "within namingContext";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-                       
-                       Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-                               "adding new values for attribute '%s'\n",
-                               at->ad->ad_cname.bv_val, 0, 0 );
-                       for ( i = 0, at_val = c_mod->sm_bvalues;
-                                       at_val->bv_val != NULL; 
-                                       i++, at_val++ ) {
-                               if ( BACKSQL_IS_ADD( at->expect_return ) ) {
-                                       pno = 1;
-                                       SQLBindParameter( sth, 1,
-                                               SQL_PARAM_OUTPUT,
-                                               SQL_C_ULONG, SQL_INTEGER,
-                                               0, 0, &prc, 0, 0);
-                               } else {
-                                       pno = 0;
-                               }
-                               po = ( BACKSQL_IS_ADD( at->param_order ) ) > 0;
-                               SQLBindParameter( sth, pno + 1 + po,
-                                       SQL_PARAM_INPUT, 
-                                       SQL_C_ULONG, SQL_INTEGER,
-                                       0, 0, &e_id->keyval, 0, 0 );
-
-                               /*
-                                * check for syntax needed here
-                                * maybe need binary bind?
-                                */
-                               SQLBindParameter( sth, pno + 2 - po,
-                                       SQL_PARAM_INPUT,
-                                       SQL_C_CHAR, SQL_CHAR,
-                                       0, 0, at_val->bv_val, 
-                                       at_val->bv_len, 0 );
-
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "executing '%s'\n", 
-                                       at->add_proc, 0, 0 );
-                               rc = SQLExecDirect( sth, at->add_proc, 
-                                               SQL_NTS );
-                               if ( rc != SQL_SUCCESS ) {
-                                       Debug( LDAP_DEBUG_TRACE,
-                                               "backsql_modify_internal(): "
-                                               "add_proc execution failed\n",
-                                               0, 0, 0 );
-                                       backsql_PrintErrors( bi->db_env,
-                                                       dbh, sth, rc );
-
-                                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                               rs->sr_err = LDAP_OTHER;
-                                               rs->sr_text = "SQL-backend error";
-                                               goto done;
-                                       }
-                               }
-#ifdef BACKSQL_REALLOC_STMT
-                               SQLFreeStmt( sth, SQL_DROP );
-                               SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-                       }
-                       break;
-                       
-               case LDAP_MOD_DELETE:
-                       if ( at->delete_proc == NULL ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "delete procedure is not defined "
-                                       "for attribute '%s'\n",
-                                       at->ad->ad_cname.bv_val, 0, 0 );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                                       rs->sr_text = "operation not permitted "
-                                               "within namingContext";
-                                       goto done;
-                               }
-
-                               break;
-                       }
-
-                       if ( c_mod->sm_bvalues == NULL ) {
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "no values given to delete "
-                                       "for attribute '%s' "
-                                       "-- deleting all values\n",
-                                       at->ad->ad_cname.bv_val, 0, 0 );
-                               goto del_all;
-                       }
-
-                       Debug( LDAP_DEBUG_TRACE, "backsql_modify_internal(): "
-                               "deleting values for attribute '%s'\n",
-                               at->ad->ad_cname.bv_val, 0, 0 );
-
-                       for ( i = 0, at_val = c_mod->sm_bvalues;
-                                       at_val->bv_val != NULL;
-                                       i++, at_val++ ) {
-                               if ( BACKSQL_IS_DEL( at->expect_return ) ) {
-                                       pno = 1;
-                                       SQLBindParameter( sth, 1,
-                                               SQL_PARAM_OUTPUT,
-                                               SQL_C_ULONG, SQL_INTEGER,
-                                               0, 0, &prc, 0, 0 );
-                               } else {
-                                       pno = 0;
-                               }
-                               po = ( BACKSQL_IS_DEL( at->param_order ) ) > 0;
-                               SQLBindParameter( sth, pno + 1 + po,
-                                       SQL_PARAM_INPUT, 
-                                       SQL_C_ULONG, SQL_INTEGER,
-                                       0, 0, &e_id->keyval, 0, 0 );
-
-                               /*
-                                * check for syntax needed here 
-                                * maybe need binary bind?
-                                */
-                               SQLBindParameter( sth, pno + 2 - po,
-                                       SQL_PARAM_INPUT, SQL_C_CHAR, SQL_CHAR,
-                                       0, 0, at_val->bv_val, 
-                                       at_val->bv_len, 0 );
-
-                               Debug( LDAP_DEBUG_TRACE,
-                                       "backsql_modify_internal(): "
-                                       "executing '%s'\n", 
-                                       at->delete_proc, 0, 0 );
-                               rc = SQLExecDirect( sth, at->delete_proc,
-                                               SQL_NTS );
-                               if ( rc != SQL_SUCCESS ) {
-                                       Debug( LDAP_DEBUG_TRACE,
-                                               "backsql_modify_internal(): "
-                                               "delete_proc execution "
-                                               "failed\n", 0, 0, 0 );
-                                       backsql_PrintErrors( bi->db_env,
-                                                       dbh, sth, rc );
-
-                                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                               rs->sr_err = LDAP_OTHER;
-                                               rs->sr_text = "SQL-backend error";
-                                               goto done;
-                                       }
-                               }
-#ifdef BACKSQL_REALLOC_STMT
-                               SQLFreeStmt( sth, SQL_DROP );
-                               SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-                       }
-                       break;
-               }
-#ifndef BACKSQL_REALLOC_STMT
-               SQLFreeStmt( sth, SQL_RESET_PARAMS );
-#else /* BACKSQL_REALLOC_STMT */
-               SQLFreeStmt( sth, SQL_DROP );
-#endif /* BACKSQL_REALLOC_STMT */
-       }
-
-done:;
-       
-#ifndef BACKSQL_REALLOC_STMT
-       SQLFreeStmt( sth, SQL_DROP );
-#endif /* BACKSQL_REALLOC_STMT */
-
-       /*
-        * FIXME: should fail in case one change fails?
-        */
-       return rs->sr_err;
-}
+#include "slap.h"
+#include "proto-sql.h"
 
 int
 backsql_modify( Operation *op, SlapReply *rs )
 {
        backsql_info            *bi = (backsql_info*)op->o_bd->be_private;
-       SQLHDBC                 dbh;
+       SQLHDBC                 dbh = SQL_NULL_HDBC;
        backsql_oc_map_rec      *oc = NULL;
-       backsql_entryID         e_id;
-       Entry                   e;
+       backsql_srch_info       bsi = { 0 };
+       Entry                   m = { 0 }, *e = NULL;
+       int                     manageDSAit = get_manageDSAit( op );
+       SQLUSMALLINT            CompletionType = SQL_ROLLBACK;
 
        /*
         * FIXME: in case part of the operation cannot be performed
         * (missing mapping, SQL write fails or so) the entire operation
         * should be rolled-back
         */
-       Debug( LDAP_DEBUG_TRACE, "==>backsql_modify(): changing entry '%s'\n",
+       Debug( LDAP_DEBUG_TRACE, "==>backsql_modify(): modifying entry \"%s\"\n",
                op->o_req_ndn.bv_val, 0, 0 );
 
        rs->sr_err = backsql_get_db_conn( op, &dbh );
        if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modify(): "
+               Debug( LDAP_DEBUG_TRACE, "   backsql_modify(): "
                        "could not get connection handle - exiting\n", 
                        0, 0, 0 );
                /*
@@ -482,1059 +59,151 @@ backsql_modify( Operation *op, SlapReply *rs )
                 */
                rs->sr_text = ( rs->sr_err == LDAP_OTHER )
                        ? "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               return 1;
+               goto done;
        }
 
-       rs->sr_err = backsql_dn2id( bi, &e_id, dbh, &op->o_req_ndn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modify(): "
-                       "could not lookup entry id\n", 0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ? "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               return 1;
-       }
+       bsi.bsi_e = &m;
+       rs->sr_err = backsql_init_search( &bsi, &op->o_req_ndn,
+                       LDAP_SCOPE_BASE, 
+                       (time_t)(-1), NULL, dbh, op, rs,
+                       slap_anlist_all_attributes,
+                       ( BACKSQL_ISF_MATCHED | BACKSQL_ISF_GET_ENTRY ) );
+       switch ( rs->sr_err ) {
+       case LDAP_SUCCESS:
+               break;
 
-       Debug( LDAP_DEBUG_TRACE, "backsql_modify(): "
-               "modifying entry '%s' (id=%ld)\n", 
-               e_id.dn.bv_val, e_id.id, 0 );
+       case LDAP_REFERRAL:
+               if ( manageDSAit && !BER_BVISNULL( &bsi.bsi_e->e_nname ) &&
+                               dn_match( &op->o_req_ndn, &bsi.bsi_e->e_nname ) )
+               {
+                       rs->sr_err = LDAP_SUCCESS;
+                       rs->sr_text = NULL;
+                       rs->sr_matched = NULL;
+                       if ( rs->sr_ref ) {
+                               ber_bvarray_free( rs->sr_ref );
+                               rs->sr_ref = NULL;
+                       }
+                       break;
+               }
+               e = &m;
+               /* fallthru */
 
-       oc = backsql_id2oc( bi, e_id.oc_id );
-       if ( oc == NULL ) {
+       default:
                Debug( LDAP_DEBUG_TRACE, "backsql_modify(): "
-                       "cannot determine objectclass of entry -- aborting\n",
+                       "could not retrieve modifyDN ID - no such entry\n", 
                        0, 0, 0 );
-               /*
-                * FIXME: should never occur, since the entry was built!!!
-                */
-
-               /*
-                * FIXME: we don't want to send back 
-                * excessively detailed messages
-                */
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
-       }
+               if ( !BER_BVISNULL( &m.e_nname ) ) {
+                       /* FIXME: should always be true! */
+                       e = &m;
 
-       e.e_attrs = NULL;
-       e.e_name = op->o_req_dn;
-       e.e_nname = op->o_req_ndn;
-       if ( !acl_check_modlist( op, &e, op->oq_modify.rs_modlist ) ) {
-               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-
-       } else {
-               rs->sr_err = backsql_modify_internal( op, rs, dbh, oc, &e_id,
-                               op->oq_modify.rs_modlist );
-       }
-
-       if ( rs->sr_err == LDAP_SUCCESS ) {
-               /*
-                * Commit only if all operations succeed
-                */
-               SQLTransact( SQL_NULL_HENV, dbh, 
-                               op->o_noop ? SQL_ROLLBACK : SQL_COMMIT );
-       }
-       send_ldap_result( op, rs );
-       Debug( LDAP_DEBUG_TRACE, "<==backsql_modify()\n", 0, 0, 0 );
-
-       return op->o_noop;
-}
-
-int
-backsql_modrdn( Operation *op, SlapReply *rs )
-{
-       backsql_info            *bi = (backsql_info*)op->o_bd->be_private;
-       SQLHDBC                 dbh;
-       SQLHSTMT                sth;
-       RETCODE                 rc;
-       backsql_entryID         e_id, pe_id, new_pid;
-       backsql_oc_map_rec      *oc = NULL;
-       struct berval           p_dn, p_ndn,
-                               *new_pdn = NULL, *new_npdn = NULL,
-                               new_dn, new_ndn;
-       LDAPRDN                 new_rdn = NULL;
-       LDAPRDN                 old_rdn = NULL;
-       Entry                   e;
-       Modifications           *mod;
-       struct berval           *newSuperior = op->oq_modrdn.rs_newSup;
-       Debug( LDAP_DEBUG_TRACE, "==>backsql_modrdn() renaming entry '%s', "
-                       "newrdn='%s', newSuperior='%s'\n",
-                       op->o_req_dn.bv_val, op->oq_modrdn.rs_newrdn.bv_val, 
-                       newSuperior ? newSuperior->bv_val : "(NULL)" );
-       rs->sr_err = backsql_get_db_conn( op, &dbh );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "could not get connection handle - exiting\n", 
-                       0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ?  "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       rs->sr_err = backsql_dn2id( bi, &e_id, dbh, &op->o_req_ndn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "could not lookup entry id\n", 0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ?  "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): entry id is %ld\n",
-               e_id.id, 0, 0 );
-
-       if ( backsql_has_children( bi, dbh, &op->o_req_ndn ) == LDAP_COMPARE_TRUE ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "entry \"%s\" has children\n",
-                       op->o_req_dn.bv_val, 0, 0 );
-               rs->sr_err = LDAP_NOT_ALLOWED_ON_NONLEAF;
-               rs->sr_text = "subtree rename not supported";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       dnParent( &op->o_req_dn, &p_dn );
-       dnParent( &op->o_req_ndn, &p_ndn );
-
-       /*
-        * namingContext "" is not supported
-        */
-       if ( p_dn.bv_len == 0 ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "parent is \"\" - aborting\n", 0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "not allowed within namingContext";
-               send_ldap_result( op, rs );
-               goto modrdn_return;
-       }
-
-       /*
-        * Check for children access to parent
-        */
-       e.e_attrs = NULL;
-       e.e_name = p_dn;
-       e.e_nname = p_ndn;
-       if ( !access_allowed( op, &e, slap_schema.si_ad_children, 
-                               NULL, ACL_WRITE, NULL ) ) {
-               Debug( LDAP_DEBUG_TRACE, "no access to parent\n", 0, 0, 0 );
-               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-               goto modrdn_return;
-       }
-
-       if ( newSuperior ) {
-               /*
-                * namingContext "" is not supported
-                */
-               if ( newSuperior->bv_len == 0 ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                               "newSuperior is \"\" - aborting\n", 0, 0, 0 );
-                       rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                       rs->sr_text = "not allowed within namingContext";
-                       send_ldap_result( op, rs );
-                       goto modrdn_return;
-               }
-
-               new_pdn = newSuperior;
-               new_npdn = op->oq_modrdn.rs_nnewSup;
-
-               e.e_name = *new_pdn;
-               e.e_nname = *new_npdn;
-
-               /*
-                * Check for children access to new parent
-                */
-               if ( !access_allowed( op, &e, slap_schema.si_ad_children, 
-                                       NULL, ACL_WRITE, NULL ) ) {
-                       Debug( LDAP_DEBUG_TRACE, "no access to new parent\n", 
-                                       0, 0, 0 );
-                       rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-                       goto modrdn_return;
+               } else {
+                       e = NULL;
                }
-
-       } else {
-               new_pdn = &p_dn;
-               new_npdn = &p_ndn;
-       }
-
-       if ( newSuperior && dn_match( &p_ndn, new_npdn ) ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "newSuperior is equal to old parent - ignored\n",
-                       0, 0, 0 );
-               newSuperior = NULL;
-       }
-
-       if ( newSuperior && dn_match( &op->o_req_ndn, new_npdn ) ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "newSuperior is equal to entry being moved "
-                       "- aborting\n", 0, 0, 0 );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "newSuperior is equal to old DN";
-               send_ldap_result( op, rs );
-               goto modrdn_return;
-       }
-
-       build_new_dn( &new_dn, new_pdn, &op->oq_modrdn.rs_newrdn );
-       rs->sr_err = dnNormalize2( NULL, &new_dn, &new_ndn, op->o_tmpmemctx );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "new dn is invalid ('%s') - aborting\n",
-                       new_dn.bv_val, 0, 0 );
-               rs->sr_text = "unable to build new DN";
-               send_ldap_result( op, rs );
-               goto modrdn_return;
+               goto done;
        }
-       
-       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): new entry dn is '%s'\n",
-                       new_dn.bv_val, 0, 0 );
 
-       rs->sr_err = backsql_dn2id( bi, &pe_id, dbh, &p_ndn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "could not lookup old parent entry id\n", 0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ? "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               goto modrdn_return;
-       }
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-               "old parent entry id is %ld\n", pe_id.id, 0, 0 );
-
-       rs->sr_err = backsql_dn2id( bi, &new_pid, dbh, new_npdn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "could not lookup new parent entry id\n", 0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ? "SQL-backend error" : NULL;
-               send_ldap_result( op, rs );
-               goto modrdn_return;
-       }
-       
-       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-               "new parent entry id is %ld\n", new_pid.id, 0, 0 );
-
-       Debug(  LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-               "executing delentry_query\n", 0, 0, 0 );
-       SQLAllocStmt( dbh, &sth );
-       SQLBindParameter( sth, 1, SQL_PARAM_INPUT, SQL_C_ULONG, SQL_INTEGER,
-                       0, 0, &e_id.id, 0, 0 );
-       rc = SQLExecDirect( sth, bi->delentry_query, SQL_NTS );
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "failed to delete record from ldap_entries\n",
-                       0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               goto modrdn_return;
-       }
-
-       SQLFreeStmt( sth, SQL_RESET_PARAMS );
+#ifdef BACKSQL_ARBITRARY_KEY
+       Debug( LDAP_DEBUG_TRACE, "   backsql_modify(): "
+               "modifying entry \"%s\" (id=%s)\n", 
+               bsi.bsi_base_id.eid_dn.bv_val,
+               bsi.bsi_base_id.eid_id.bv_val, 0 );
+#else /* ! BACKSQL_ARBITRARY_KEY */
+       Debug( LDAP_DEBUG_TRACE, "   backsql_modify(): "
+               "modifying entry \"%s\" (id=%ld)\n", 
+               bsi.bsi_base_id.eid_dn.bv_val, bsi.bsi_base_id.eid_id, 0 );
+#endif /* ! BACKSQL_ARBITRARY_KEY */
 
-       Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-               "executing insentry_query\n", 0, 0, 0 );
-       backsql_BindParamStr( sth, 1, new_dn.bv_val, BACKSQL_MAX_DN_LEN );
-       SQLBindParameter( sth, 2, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &e_id.oc_id, 0, 0 );
-       SQLBindParameter( sth, 3, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &new_pid.id, 0, 0 );
-       SQLBindParameter( sth, 4, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &e_id.keyval, 0, 0 );
-       rc = SQLExecDirect( sth, bi->insentry_query, SQL_NTS );
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_modrdn(): "
-                       "could not insert ldap_entries record\n", 0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               goto modrdn_return;
+       if ( get_assert( op ) &&
+                       ( test_filter( op, &m, get_assertion( op ) )
+                         != LDAP_COMPARE_TRUE ))
+       {
+               rs->sr_err = LDAP_ASSERTION_FAILED;
+               e = &m;
+               goto done;
        }
 
-       /*
-        * Get attribute type and attribute value of our new rdn,
-        * we will need to add that to our new entry
-        */
-       if ( ldap_bv2rdn( &op->oq_modrdn.rs_newrdn, &new_rdn,
-                               (char **)&rs->sr_text, 
-                               LDAP_DN_FORMAT_LDAP ) ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG ( OPERATION, ERR, 
-                       "backsql_modrdn: can't figure out "
-                       "type(s)/values(s) of newrdn\n", 
-                       0, 0, 0 );
-#else
-               Debug( LDAP_DEBUG_TRACE,
-                       "backsql_modrdn: can't figure out "
-                       "type(s)/values(s) of newrdn\n", 
-                       0, 0, 0 );
-#endif
-               rs->sr_err = LDAP_INVALID_DN_SYNTAX;
-               goto modrdn_return;
-       }
-
-#ifdef NEW_LOGGING
-       LDAP_LOG ( OPERATION, RESULTS, 
-               "backsql_modrdn: new_rdn_type=\"%s\", "
-               "new_rdn_val=\"%s\"\n",
-               new_rdn[ 0 ]->la_attr.bv_val, 
-               new_rdn[ 0 ]->la_value.bv_val, 0 );
-#else
-       Debug( LDAP_DEBUG_TRACE,
-               "backsql_modrdn: new_rdn_type=\"%s\", "
-               "new_rdn_val=\"%s\"\n",
-               new_rdn[ 0 ]->la_attr.bv_val,
-               new_rdn[ 0 ]->la_value.bv_val, 0 );
-#endif
+       oc = backsql_id2oc( bi, bsi.bsi_base_id.eid_oc_id );
+       assert( oc != NULL );
 
-       if ( op->oq_modrdn.rs_deleteoldrdn ) {
-               if ( ldap_bv2rdn( &op->o_req_dn, &old_rdn,
-                                       (char **)&rs->sr_text,
-                                       LDAP_DN_FORMAT_LDAP ) ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG ( OPERATION, ERR, 
-                               "backsql_modrdn: can't figure out "
-                               "type(s)/values(s) of old_rdn\n", 
-                               0, 0, 0 );
-#else
-                       Debug( LDAP_DEBUG_TRACE,
-                               "backsql_modrdn: can't figure out "
-                               "the old_rdn type(s)/value(s)\n", 
-                               0, 0, 0 );
-#endif
-                       rs->sr_err = LDAP_OTHER;
-                       goto modrdn_return;             
-               }
+       if ( !acl_check_modlist( op, &m, op->oq_modify.rs_modlist ) ) {
+               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
+               e = &m;
+               goto done;
        }
 
-       e.e_name = new_dn;
-       e.e_nname = new_ndn;
-       rs->sr_err = slap_modrdn2mods( op, rs, &e, old_rdn, new_rdn, &mod );
+       rs->sr_err = backsql_modify_internal( op, rs, dbh, oc,
+                       &bsi.bsi_base_id, op->oq_modify.rs_modlist );
        if ( rs->sr_err != LDAP_SUCCESS ) {
-               goto modrdn_return;
+               e = &m;
+               goto do_transact;
        }
 
-       if ( !acl_check_modlist( op, &e, mod )) {
-               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-               goto modrdn_return;
-       }
-
-       oc = backsql_id2oc( bi, e_id.oc_id );
-       rs->sr_err = backsql_modify_internal( op, rs, dbh, oc, &e_id, mod );
-
-       if ( rs->sr_err == LDAP_SUCCESS ) {
-
-               /*
-                * Commit only if all operations succeed
-                */
-               SQLTransact( SQL_NULL_HENV, dbh,
-                               op->o_noop ? SQL_ROLLBACK : SQL_COMMIT );
-       }
+       if ( BACKSQL_CHECK_SCHEMA( bi ) ) {
+               char            textbuf[ SLAP_TEXT_BUFLEN ] = { '\0' };
 
-modrdn_return:
-       SQLFreeStmt( sth, SQL_DROP );
+               backsql_entry_clean( op, &m );
 
-       if ( new_dn.bv_val ) {
-               ch_free( new_dn.bv_val );
-       }
-       
-       if ( new_ndn.bv_val ) {
-               ch_free( new_ndn.bv_val );
-       }
-       
-       /* LDAP v2 supporting correct attribute handling. */
-       if ( new_rdn != NULL ) {
-               ldap_rdnfree( new_rdn );
-       }
-       if ( old_rdn != NULL ) {
-               ldap_rdnfree( old_rdn );
-       }
-       if( mod != NULL ) {
-               Modifications *tmp;
-               for (; mod; mod=tmp ) {
-                       tmp = mod->sml_next;
-                       free( mod );
+               bsi.bsi_e = &m;
+               rs->sr_err = backsql_id2entry( &bsi, &bsi.bsi_base_id );
+               if ( rs->sr_err != LDAP_SUCCESS ) {
+                       e = &m;
+                       goto do_transact;
                }
-       }
-
-       send_ldap_result( op, rs );
-
-       Debug( LDAP_DEBUG_TRACE, "<==backsql_modrdn()\n", 0, 0, 0 );
-       return op->o_noop;
-}
-
-int
-backsql_add( Operation *op, SlapReply *rs )
-{
-       backsql_info            *bi = (backsql_info*)op->o_bd->be_private;
-       SQLHDBC                 dbh;
-       SQLHSTMT                sth;
-       unsigned long           new_keyval = 0;
-       long                    i;
-       RETCODE                 rc;
-       backsql_oc_map_rec      *oc = NULL;
-       backsql_at_map_rec      *at_rec = NULL;
-       backsql_entryID         e_id, parent_id;
-       Entry                   p;
-       Attribute               *at;
-       struct berval           *at_val;
-       struct berval           pdn;
-       /* first parameter #, parameter order */
-       SQLUSMALLINT            pno, po;
-       /* procedure return code */
-       int                     prc;
 
-       Debug( LDAP_DEBUG_TRACE, "==>backsql_add(): adding entry '%s'\n",
-                       op->oq_add.rs_e->e_name.bv_val, 0, 0 );
-
-       /* check schema */
-       if ( global_schemacheck ) {
-               char            textbuf[ SLAP_TEXT_BUFLEN ] = { '\0' };
-
-               rs->sr_err = entry_schema_check( op->o_bd, op->oq_add.rs_e,
-                               NULL,
-                               &rs->sr_text, textbuf, sizeof( textbuf ) );
+               rs->sr_err = entry_schema_check( op, &m, NULL, 0,
+                       &rs->sr_text, textbuf, sizeof( textbuf ) );
                if ( rs->sr_err != LDAP_SUCCESS ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
+                       Debug( LDAP_DEBUG_TRACE, "   backsql_add(\"%s\"): "
                                "entry failed schema check -- aborting\n",
-                               0, 0, 0 );
-                       send_ldap_result( op, rs );
-                       return 1;
+                               m.e_name.bv_val, 0, 0 );
+                       e = NULL;
+                       goto do_transact;
                }
        }
 
-       /* search structural objectClass */
-       for ( at = op->oq_add.rs_e->e_attrs; at != NULL; at = at->a_next ) {
-               if ( at->a_desc == slap_schema.si_ad_structuralObjectClass ) {
-                       break;
-               }
-       }
-
-       /* there must exist */
-       assert( at != NULL );
-
-       /* I guess we should play with sub/supertypes to find a suitable oc */
-       oc = backsql_name2oc( bi, &at->a_vals[0] );
-
-       if ( oc == NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "cannot determine objectclass of entry -- aborting\n",
-                       0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "operation not permitted within namingContext";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       if ( oc->create_proc == NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "create procedure is not defined for this objectclass "
-                       "- aborting\n", 0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "operation not permitted within namingContext";
-               send_ldap_result( op, rs );
-               return 1;
-
-       } else if ( BACKSQL_CREATE_NEEDS_SELECT( bi )
-                       && oc->create_keyval == NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "create procedure needs select procedure, "
-                       "but none is defined - aborting\n", 0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "operation not permitted within namingContext";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       rs->sr_err = backsql_get_db_conn( op, &dbh );
-       if ( prc != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "could not get connection handle - exiting\n", 
-                       0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ?  "SQL-backend error" : NULL,
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
+do_transact:;
        /*
-        * Check if entry exists
-        */
-       rs->sr_err = backsql_dn2id( bi, &e_id, dbh, &op->oq_add.rs_e->e_name );
-       if ( rs->sr_err == LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "entry '%s' exists\n",
-                       op->oq_add.rs_e->e_name.bv_val, 0, 0 );
-               rs->sr_err = LDAP_ALREADY_EXISTS;
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       /*
-        * Check if parent exists
-        */
-       dnParent( &op->oq_add.rs_e->e_name, &pdn );
-       rs->sr_err = backsql_dn2id( bi, &parent_id, dbh, &pdn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "could not lookup parent entry for new record '%s'\n",
-                       pdn.bv_val, 0, 0 );
-
-               if ( rs->sr_err != LDAP_NO_SUCH_OBJECT ) {
-                       send_ldap_result( op, rs );
-                       return 1;
-               }
-
-               /*
-                * Look for matched
-                */
-               while ( 1 ) {
-                       struct berval   dn;
-                       char            *matched = NULL;
-
-                       dn = pdn;
-                       dnParent( &dn, &pdn );
-
-                       /*
-                        * Empty DN ("") defaults to LDAP_SUCCESS
-                        */
-                       rs->sr_err = backsql_dn2id( bi, &parent_id, dbh, &pdn );
-                       switch ( rs->sr_err ) {
-                       case LDAP_NO_SUCH_OBJECT:
-                               if ( pdn.bv_len > 0 ) {
-                                       break;
-                               }
-                               /* fail over to next case */
-                               
-                       case LDAP_SUCCESS:
-                               matched = pdn.bv_val;
-                               /* fail over to next case */
-
-                       default:
-                               rs->sr_err = LDAP_NO_SUCH_OBJECT;
-                               rs->sr_matched = matched;
-                               send_ldap_result( op, rs );
-                               return 1;
-                       } 
-               }
-       }
-
-       /*
-        * create_proc is executed; if expect_return is set, then
-        * an output parameter is bound, which should contain 
-        * the id of the added row; otherwise the procedure
-        * is expected to return the id as the first column of a select
+        * Commit only if all operations succeed
         */
-
-       p.e_attrs = NULL;
-       p.e_name = pdn;
-       dnParent( &op->oq_add.rs_e->e_nname, &p.e_nname );
-       if ( !access_allowed( op, &p, slap_schema.si_ad_children,
-                               NULL, ACL_WRITE, NULL ) ) {
-               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       rc = SQLAllocStmt( dbh, &sth );
-       if ( rc != SQL_SUCCESS ) {
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
+       if ( rs->sr_err == LDAP_SUCCESS && !op->o_noop ) {
+               CompletionType = SQL_COMMIT;
        }
 
-       if ( BACKSQL_IS_ADD( oc->expect_return ) ) {
-               SQLBindParameter( sth, 1, SQL_PARAM_OUTPUT, SQL_C_ULONG, 
-                               SQL_INTEGER, 0, 0, &new_keyval, 0, 0 );
-       }
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_add(): executing '%s'\n",
-               oc->create_proc, 0, 0 );
-       rc = SQLExecDirect( sth, oc->create_proc, SQL_NTS );
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "create_proc execution failed\n", 0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc);
-               SQLFreeStmt( sth, SQL_DROP );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-       if ( op->o_noop ) {
-               SQLTransact( SQL_NULL_HENV, dbh, SQL_ROLLBACK );
-       }
-
-       if ( !BACKSQL_IS_ADD( oc->expect_return ) ) {
-               SWORD           ncols;
-               SQLINTEGER      value_len;
-
-               if ( BACKSQL_CREATE_NEEDS_SELECT( bi ) ) {
-#ifndef BACKSQL_REALLOC_STMT
-                       SQLFreeStmt( sth, SQL_RESET_PARAMS );
-#else /* BACKSQL_REALLOC_STMT */
-                       SQLFreeStmt( sth, SQL_DROP );
-                       rc = SQLAllocStmt( dbh, &sth );
-                       if ( rc != SQL_SUCCESS ) {
-                               rs->sr_err = LDAP_OTHER;
-                               rs->sr_text = "SQL-backend error";
-                               send_ldap_result( op, rs );
-                               return 1;
-                       }
-#endif /* BACKSQL_REALLOC_STMT */
-
-                       rc = SQLExecDirect( sth, oc->create_keyval, SQL_NTS );
-                       if ( rc != SQL_SUCCESS ) {
-                               rs->sr_err = LDAP_OTHER;
-                               rs->sr_text = "SQL-backend error";
-                               send_ldap_result( op, rs );
-                               return 1;
-                       }
-               }
-
-               /*
-                * the query to know the id of the inserted entry
-                * must be embedded in the create procedure
-                */
-               rc = SQLNumResultCols( sth, &ncols );
-               if ( rc != SQL_SUCCESS ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "create_proc result evaluation failed\n",
-                               0, 0, 0 );
-                       backsql_PrintErrors( bi->db_env, dbh, sth, rc);
-                       SQLFreeStmt( sth, SQL_DROP );
-                       rs->sr_err = LDAP_OTHER;
-                       rs->sr_text = "SQL-backend error";
-                       send_ldap_result( op, rs );
-                       return 1;
-
-               } else if ( ncols != 1 ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "create_proc result is bogus (ncols=%d)\n",
-                               ncols, 0, 0 );
-                       backsql_PrintErrors( bi->db_env, dbh, sth, rc);
-                       SQLFreeStmt( sth, SQL_DROP );
-                       rs->sr_err = LDAP_OTHER;
-                       rs->sr_text = "SQL-backend error";
-                       send_ldap_result( op, rs );
-                       return 1;
-               }
+       SQLTransact( SQL_NULL_HENV, dbh, CompletionType );
 
-#if 0
+done:;
+#ifdef SLAP_ACL_HONOR_DISCLOSE
+       if ( e != NULL ) {
+               if ( !access_allowed( op, e, slap_schema.si_ad_entry, NULL,
+                                       ACL_DISCLOSE, NULL ) )
                {
-                       SQLCHAR         colname[ 64 ];
-                       SQLSMALLINT     name_len, col_type, col_scale, col_null;
-                       UDWORD          col_prec;
-
-                       /*
-                        * FIXME: check whether col_type is compatible,
-                        * if it can be null and so on ...
-                        */
-                       rc = SQLDescribeCol( sth, (SQLUSMALLINT)1, 
-                                       &colname[ 0 ], 
-                                       (SQLUINTEGER)( sizeof( colname ) - 1 ),
-                                       &name_len, &col_type,
-                                       &col_prec, &col_scale, &col_null );
-               }
-#endif
-
-               rc = SQLBindCol( sth, (SQLUSMALLINT)1, SQL_C_ULONG,
-                               (SQLPOINTER)&new_keyval, 
-                               (SQLINTEGER)sizeof( new_keyval ), 
-                               &value_len );
-
-               rc = SQLFetch( sth );
-
-               if ( value_len <= 0 ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "create_proc result is empty?\n",
-                               0, 0, 0 );
-                       backsql_PrintErrors( bi->db_env, dbh, sth, rc);
-                       SQLFreeStmt( sth, SQL_DROP );
-                       rs->sr_err = LDAP_OTHER;
-                       rs->sr_text = "SQL-backend error";
-                       send_ldap_result( op, rs );
-                       return 1;
-               }
-       }
-
-#ifndef BACKSQL_REALLOC_STMT
-       SQLFreeStmt( sth, SQL_RESET_PARAMS );
-#else /* BACKSQL_REALLOC_STMT */
-       SQLFreeStmt( sth, SQL_DROP );
-#endif /* BACKSQL_REALLOC_STMT */
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-               "create_proc returned keyval=%ld\n", new_keyval, 0, 0 );
-
-       for ( at = op->oq_add.rs_e->e_attrs; at != NULL; at = at->a_next ) {
-               SQLUSMALLINT    currpos;
-
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "adding attribute '%s'\n", 
-                       at->a_desc->ad_cname.bv_val, 0, 0 );
-
-               /*
-                * Skip:
-                * - the first occurrence of objectClass, which is used
-                *   to determine how to bulid the SQL entry (FIXME ?!?)
-                * - operational attributes
-                *   empty attributes (FIXME ?!?)
-                */
-               if ( backsql_attr_skip( at->a_desc, at->a_vals ) ) {
-                       continue;
-               }
-
-               at_rec = backsql_ad2at( oc, at->a_desc ); 
-  
-               if ( at_rec == NULL ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "attribute '%s' is not registered "
-                               "in objectclass '%s'\n",
-                               at->a_desc->ad_cname.bv_val,
-                               BACKSQL_OC_NAME( oc ), 0 );
-
-                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                               rs->sr_text = "operation not permitted "
-                                       "within namingContext";
-                               send_ldap_result( op, rs );
-                               return 1;
-                       }
-
-                       continue;
-               }
-               
-               if ( at_rec->add_proc == NULL ) {
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "add procedure is not defined "
-                               "for attribute '%s'\n",
-                               at->a_desc->ad_cname.bv_val, 0, 0 );
-
-                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-                               rs->sr_text = "operation not permitted "
-                                       "within namingContext";
-                               send_ldap_result( op, rs );
-                               return 1;
-                       }
-
-                       continue;
-               }
-
-#ifdef BACKSQL_REALLOC_STMT
-               rc = backsql_Prepare( dbh, &sth, at_rec->add_proc, 0 );
-               if ( rc != SQL_SUCCESS ) {
-
-                       if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                               rs->sr_err = LDAP_OTHER;
-                               rs->sr_text = "SQL-backend error";
-                               send_ldap_result( op, rs );
-                               return 1;
-                       }
-
-                       continue;
-               }
-#endif /* BACKSQL_REALLOC_STMT */
-
-               if ( BACKSQL_IS_ADD( at_rec->expect_return ) ) {
-                       pno = 1;
-                       SQLBindParameter( sth, 1, SQL_PARAM_OUTPUT,
-                                       SQL_C_ULONG, SQL_INTEGER,
-                                       0, 0, &prc, 0, 0 );
-               } else {
-                       pno = 0;
-               }
-
-               po = ( BACKSQL_IS_ADD( at_rec->param_order ) ) > 0;
-               currpos = pno + 1 + po;
-               SQLBindParameter( sth, currpos,
-                               SQL_PARAM_INPUT, SQL_C_ULONG,
-                               SQL_INTEGER, 0, 0, &new_keyval, 0, 0 );
-               currpos = pno + 2 - po;
-
-               for ( i = 0, at_val = &at->a_vals[ i ];
-                               at_val->bv_val != NULL;
-                               i++, at_val = &at->a_vals[ i ] ) {
-
-                       /*
-                        * Do not deal with the objectClass that is used
-                        * to build the entry
-                        */
-                       if ( at->a_desc == slap_schema.si_ad_objectClass ) {
-                               if ( bvmatch( at_val, &oc->oc->soc_cname ) ) {
-                                       continue;
-                               }
-                       }
-
-                       /*
-                        * check for syntax needed here 
-                        * maybe need binary bind?
-                        */
-
-                       backsql_BindParamStr( sth, currpos,
-                                       at_val->bv_val, at_val->bv_len + 1 );
-#ifdef SECURITY_PARANOID
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "executing '%s', id=%ld\n", 
-                               at_rec->add_proc, new_keyval, 0 );
-#else
-                       Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                               "executing '%s' with val='%s', id=%ld\n", 
-                               at_rec->add_proc, at_val->bv_val, new_keyval );
-#endif
-#ifndef BACKSQL_REALLOC_STMT
-                       rc = SQLExecDirect( sth, at_rec->add_proc, SQL_NTS );
-#else /* BACKSQL_REALLOC_STMT */
-                       rc = SQLExecute( sth );
-#endif /* BACKSQL_REALLOC_STMT */
-                       if ( rc != SQL_SUCCESS ) {
-                               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                                       "add_proc execution failed\n", 
-                                       0, 0, 0 );
-                               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-
-                               if ( BACKSQL_FAIL_IF_NO_MAPPING( bi ) ) {
-                                       rs->sr_err = LDAP_OTHER;
-                                       rs->sr_text = "SQL-backend error";
-                                       send_ldap_result( op, rs );
-                                       return 1;
-                               }
+                       rs->sr_err = LDAP_NO_SUCH_OBJECT;
+                       rs->sr_text = NULL;
+                       rs->sr_matched = NULL;
+                       if ( rs->sr_ref ) {
+                               ber_bvarray_free( rs->sr_ref );
+                               rs->sr_ref = NULL;
                        }
                }
-#ifndef BACKSQL_REALLOC_STMT
-               SQLFreeStmt( sth, SQL_RESET_PARAMS ); 
-#else /* BACKSQL_REALLOC_STMT */
-               SQLFreeStmt( sth, SQL_DROP );
-#endif /* BACKSQL_REALLOC_STMT */
        }
-
-#ifdef BACKSQL_REALLOC_STMT
-       rc = backsql_Prepare( dbh, &sth, bi->insentry_query, 0 );
-       if ( rc != SQL_SUCCESS ) {
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-#endif /* BACKSQL_REALLOC_STMT */
-       
-       backsql_BindParamStr( sth, 1, op->oq_add.rs_e->e_name.bv_val,
-                       BACKSQL_MAX_DN_LEN );
-       SQLBindParameter( sth, 2, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &oc->id, 0, 0 );
-       SQLBindParameter( sth, 3, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &parent_id.id, 0, 0 );
-       SQLBindParameter( sth, 4, SQL_PARAM_INPUT, SQL_C_LONG, SQL_INTEGER,
-                       0, 0, &new_keyval, 0, 0 );
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_add(): executing '%s' for dn '%s'\n",
-                       bi->insentry_query, op->oq_add.rs_e->e_name.bv_val, 0 );
-       Debug( LDAP_DEBUG_TRACE, " for oc_map_id=%ld, parent_id=%ld, "
-                       "keyval=%ld\n", oc->id, parent_id.id, new_keyval );
-#ifndef BACKSQL_REALLOC_STMT
-       rc = SQLExecDirect( sth, bi->insentry_query, SQL_NTS );
-#else /* BACKSQL_REALLOC_STMT */
-       rc = SQLExecute( sth );
-#endif /* BACKSQL_REALLOC_STMT */
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_add(): "
-                       "could not insert ldap_entries record\n", 0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-               
-               /*
-                * execute delete_proc to delete data added !!!
-                */
-               SQLFreeStmt( sth, SQL_DROP );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-       
-       SQLFreeStmt( sth, SQL_DROP );
-
-       /*
-        * Commit only if all operations succeed
-        */
-       SQLTransact( SQL_NULL_HENV, dbh, 
-                       op->o_noop ? SQL_ROLLBACK : SQL_COMMIT );
-
-       /*
-        * FIXME: NOOP does not work for add -- it works for all 
-        * the other operations, and I don't get the reason :(
-        */
+#endif /* SLAP_ACL_HONOR_DISCLOSE */
 
        send_ldap_result( op, rs );
 
-       return op->o_noop;
-}
-
-int
-backsql_delete( Operation *op, SlapReply *rs )
-{
-       backsql_info            *bi = (backsql_info*)op->o_bd->be_private;
-       SQLHDBC                 dbh;
-       SQLHSTMT                sth;
-       RETCODE                 rc;
-       backsql_oc_map_rec      *oc = NULL;
-       backsql_entryID         e_id;
-       Entry                   e;
-       /* first parameter no */
-       SQLUSMALLINT            pno;
-
-       Debug( LDAP_DEBUG_TRACE, "==>backsql_delete(): deleting entry '%s'\n",
-                       op->o_req_ndn.bv_val, 0, 0 );
-
-       dnParent( &op->o_req_dn, &e.e_name );
-       dnParent( &op->o_req_ndn, &e.e_nname );
-       e.e_attrs = NULL;
-
-       /* check parent for "children" acl */
-       if ( !access_allowed( op, &e, slap_schema.si_ad_children, 
-                       NULL, ACL_WRITE, NULL ) ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "no write access to parent\n", 
-                       0, 0, 0 );
-               rs->sr_err = LDAP_INSUFFICIENT_ACCESS;
-               send_ldap_result( op, rs );
-               return 1;
-
-       }
-       
-       rs->sr_err = backsql_get_db_conn( op, &dbh );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "could not get connection handle - exiting\n", 
-                       0, 0, 0 );
-               rs->sr_text = ( rs->sr_err == LDAP_OTHER )
-                       ? "SQL-backend error" : NULL,
-               send_ldap_result( op, rs );
-               return 1;
-       }
-       
-       rs->sr_err = backsql_dn2id( bi, &e_id, dbh, &op->o_req_ndn );
-       if ( rs->sr_err != LDAP_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "could not lookup entry id\n", 0, 0, 0 );
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       rs->sr_err = backsql_has_children( bi, dbh, &op->o_req_ndn );
-       switch ( rs->sr_err ) {
-       case LDAP_COMPARE_TRUE:
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "entry \"%s\" has children\n",
-                       op->o_req_dn.bv_val, 0, 0 );
-               rs->sr_err = LDAP_NOT_ALLOWED_ON_NONLEAF;
-               rs->sr_text = "subtree delete not supported";
-               send_ldap_result( op, rs );
-               return 1;
-
-       case LDAP_COMPARE_FALSE:
-               break;
-
-       default:
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       oc = backsql_id2oc( bi, e_id.oc_id );
-       if ( oc == NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "cannot determine objectclass of entry -- aborting\n",
-                       0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "operation not permitted within namingContext";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-
-       if ( oc->delete_proc == NULL ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "delete procedure is not defined "
-                       "for this objectclass - aborting\n", 0, 0, 0 );
-               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
-               rs->sr_text = "operation not permitted within namingContext";
-               send_ldap_result( op, rs );
-               return 1;
+       if ( !BER_BVISNULL( &bsi.bsi_base_id.eid_ndn ) ) {
+               (void)backsql_free_entryID( op, &bsi.bsi_base_id, 0 );
        }
 
-       SQLAllocStmt( dbh, &sth );
-       if ( BACKSQL_IS_DEL( oc->expect_return ) ) {
-               pno = 1;
-               SQLBindParameter( sth, 1, SQL_PARAM_OUTPUT, SQL_C_ULONG,
-                               SQL_INTEGER, 0, 0, &rc, 0, 0 );
-       } else {
-               pno = 0;
+       if ( !BER_BVISNULL( &m.e_nname ) ) {
+               backsql_entry_clean( op, &m );
        }
 
-       SQLBindParameter( sth, pno + 1, SQL_PARAM_INPUT, 
-                       SQL_C_ULONG, SQL_INTEGER, 0, 0, &e_id.keyval, 0, 0 );
-
-       Debug( LDAP_DEBUG_TRACE, "backsql_delete(): executing '%s'\n",
-                       oc->delete_proc, 0, 0 );
-       rc = SQLExecDirect( sth, oc->delete_proc, SQL_NTS );
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "delete_proc execution failed\n", 0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-               SQLFreeStmt( sth, SQL_DROP );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
-       }
-#ifndef BACKSQL_REALLOC_STMT
-       SQLFreeStmt( sth, SQL_RESET_PARAMS );
-#else /* BACKSQL_REALLOC_STMT */
-       SQLFreeStmt( sth, SQL_DROP );
-       SQLAllocStmt( dbh, &sth );
-#endif /* BACKSQL_REALLOC_STMT */
-
-       SQLBindParameter( sth, 1, SQL_PARAM_INPUT, SQL_C_ULONG, SQL_INTEGER,
-                       0, 0, &e_id.id, 0, 0 );
-       rc = SQLExecDirect( sth, bi->delentry_query, SQL_NTS );
-       if ( rc != SQL_SUCCESS ) {
-               Debug( LDAP_DEBUG_TRACE, "backsql_delete(): "
-                       "failed to delete record from ldap_entries\n", 
-                       0, 0, 0 );
-               backsql_PrintErrors( bi->db_env, dbh, sth, rc );
-               SQLFreeStmt( sth, SQL_DROP );
-               rs->sr_err = LDAP_OTHER;
-               rs->sr_text = "SQL-backend error";
-               send_ldap_result( op, rs );
-               return 1;
+       if ( bsi.bsi_attrs != NULL ) {
+               op->o_tmpfree( bsi.bsi_attrs, op->o_tmpmemctx );
        }
-       
-       SQLFreeStmt( sth, SQL_DROP );
 
-       /*
-        * Commit only if all operations succeed
-        *
-        * FIXME: backsql_add() does not fail if add operations 
-        * are not available for some attributes, or if
-        * a multiple value add actually results in a replace, 
-        * or if a single operation on an attribute fails 
-        * for any reason
-        */
-       SQLTransact( SQL_NULL_HENV, dbh, 
-                       op->o_noop ? SQL_ROLLBACK : SQL_COMMIT );
+       Debug( LDAP_DEBUG_TRACE, "<==backsql_modify()\n", 0, 0, 0 );
 
-       send_ldap_result( op, rs );
-       Debug( LDAP_DEBUG_TRACE, "<==backsql_delete()\n", 0, 0, 0 );
-       return op->o_noop;
+       return rs->sr_err;
 }
 
-#endif /* SLAPD_SQL */
-