Fix previous idl_find patch. Cannot test <=0 with unsigned ints, duh...

[openldap] / servers / slapd / add.c

diff --git a/servers/slapd/add.c b/servers/slapd/add.c
index bb549fba3904953d6e5d953fecbb7ca4e177688d..31eed448a9362b703c0ad8385302583004c9e4e4 100644 (file)
--- a/servers/slapd/add.c
+++ b/servers/slapd/add.c
@@ -47,8 +47,8 @@ do_add( Connection *conn, Operation *op )
        int     manageDSAit;
 
 #ifdef NEW_LOGGING
-        LDAP_LOG(( "operation", LDAP_LEVEL_ENTRY,
-                   "do_add: conn %d enter\n", conn->c_connid ));
+       LDAP_LOG(( "operation", LDAP_LEVEL_ENTRY,
+                  "do_add: conn %d enter\n", conn->c_connid ));
 #else
        Debug( LDAP_DEBUG_TRACE, "do_add\n", 0, 0, 0 );
 #endif
@@ -67,8 +67,8 @@ do_add( Connection *conn, Operation *op )
        /* get the name */
        if ( ber_scanf( ber, "{a", /*}*/ &dn ) == LBER_ERROR ) {
 #ifdef NEW_LOGGING
-            LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
-                       "do_add: conn %d ber_scanf failed\n", conn->c_connid ));
+               LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
+                          "do_add: conn %d ber_scanf failed\n", conn->c_connid ));
 #else
                Debug( LDAP_DEBUG_ANY, "do_add: ber_scanf failed\n", 0, 0, 0 );
 #endif
@@ -81,8 +81,8 @@ do_add( Connection *conn, Operation *op )
 
        if ( dn_normalize( ndn ) == NULL ) {
 #ifdef NEW_LOGGING
-            LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
-                       "do_add: conn %d  invalid dn (%s)\n", conn->c_connid, dn ));
+               LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
+                          "do_add: conn %d      invalid dn (%s)\n", conn->c_connid, dn ));
 #else
                Debug( LDAP_DEBUG_ANY, "do_add: invalid dn (%s)\n", dn, 0, 0 );
 #endif
@@ -101,8 +101,8 @@ do_add( Connection *conn, Operation *op )
        e->e_private = NULL;
 
 #ifdef NEW_LOGGING
-        LDAP_LOG(( "operation", LDAP_LEVEL_ARGS,
-                   "do_add: conn %d  ndn (%s)\n", conn->c_connid, e->e_ndn ));
+       LDAP_LOG(( "operation", LDAP_LEVEL_ARGS,
+                  "do_add: conn %d  ndn (%s)\n", conn->c_connid, e->e_ndn ));
 #else
        Debug( LDAP_DEBUG_ARGS, "do_add: ndn (%s)\n", e->e_ndn, 0, 0 );
 #endif
@@ -119,8 +119,8 @@ do_add( Connection *conn, Operation *op )
 
                if ( rc == LBER_ERROR ) {
 #ifdef NEW_LOGGING
-                    LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
-                               "do_add: conn %d  decoding error \n", conn->c_connid ));
+                       LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
+                                  "do_add: conn %d      decoding error \n", conn->c_connid ));
 #else
                        Debug( LDAP_DEBUG_ANY, "do_add: decoding error\n", 0, 0, 0 );
 #endif
@@ -133,9 +133,9 @@ do_add( Connection *conn, Operation *op )
 
                if ( mod->ml_bvalues == NULL ) {
 #ifdef NEW_LOGGING
-                    LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
-                               "do_add: conn %d  no values for type %s\n",
-                               conn->c_connid, mod->ml_type ));
+                       LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
+                                  "do_add: conn %d      no values for type %s\n",
+                                  conn->c_connid, mod->ml_type ));
 #else
                        Debug( LDAP_DEBUG_ANY, "no values for type %s\n",
                                mod->ml_type, 0, 0 );
@@ -153,8 +153,8 @@ do_add( Connection *conn, Operation *op )
 
        if ( ber_scanf( ber, /*{*/ "}") == LBER_ERROR ) {
 #ifdef NEW_LOGGING
-            LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
-                       "do_add: conn %d  ber_scanf failed\n", conn->c_connid ));
+               LDAP_LOG(( "operation", LDAP_LEVEL_ERR,
+                          "do_add: conn %d      ber_scanf failed\n", conn->c_connid ));
 #else
                Debug( LDAP_DEBUG_ANY, "do_add: ber_scanf failed\n", 0, 0, 0 );
 #endif
@@ -166,8 +166,8 @@ do_add( Connection *conn, Operation *op )
 
        if( (rc = get_ctrls( conn, op, 1 )) != LDAP_SUCCESS ) {
 #ifdef NEW_LOGGING
-            LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
-                       "do_add: conn %d  get_ctrls failed\n", conn->c_connid ));
+               LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
+                          "do_add: conn %d      get_ctrls failed\n", conn->c_connid ));
 #else
                Debug( LDAP_DEBUG_ANY, "do_add: get_ctrls failed\n", 0, 0, 0 );
 #endif
@@ -190,8 +190,7 @@ do_add( Connection *conn, Operation *op )
                goto done;
        }
 
-       manageDSAit = get_manageDSAit( op ) &&
-               is_entry_referral( e );
+       manageDSAit = get_manageDSAit( op );
 
        /*
         * We could be serving multiple database backends.  Select the
@@ -227,28 +226,27 @@ do_add( Connection *conn, Operation *op )
         */
        if ( be->be_add ) {
                /* do the update here */
-#ifdef SLAPD_MULTIMASTER
-               if ( (be->be_lastmod == ON || (be->be_lastmod == UNDEFINED &&
-                       global_lastmod == ON)) && (be->be_update_ndn == NULL ||
-                       strcmp( be->be_update_ndn, op->o_ndn )) )
-#else
-               if ( be->be_update_ndn == NULL ||
-                       strcmp( be->be_update_ndn, op->o_ndn ) == 0 )
+               int repl_user = (be->be_update_ndn != NULL &&
+                       strcmp( be->be_update_ndn, op->o_ndn ) == 0);
+#ifndef SLAPD_MULTIMASTER
+               if ( be->be_update_ndn == NULL || repl_user )
 #endif
                {
                        int update = be->be_update_ndn != NULL;
+                       char textbuf[SLAP_TEXT_BUFLEN];
+                       size_t textlen = sizeof textbuf;
+
+                       rc = slap_modlist2mods( modlist, update, &mods, &text,
+                               textbuf, textlen );
 
-                       rc = slap_modlist2mods( modlist, update, &mods, &text );
                        if( rc != LDAP_SUCCESS ) {
                                send_ldap_result( conn, op, rc,
                                        NULL, text, NULL, NULL );
                                goto done;
                        }
 
-#ifndef SLAPD_MULTIMASTER
                        if ( (be->be_lastmod == ON || (be->be_lastmod == UNDEFINED &&
-                               global_lastmod == ON)) && !update )
-#endif
+                               global_lastmod == ON)) && !repl_user )
                        {
                                Modifications **modstail;
                                for( modstail = &mods;
@@ -275,8 +273,7 @@ do_add( Connection *conn, Operation *op )
 
                        if ( (*be->be_add)( be, conn, op, e ) == 0 ) {
 #ifdef SLAPD_MULTIMASTER
-                               if (be->be_update_ndn == NULL ||
-                                       strcmp( be->be_update_ndn, op->o_ndn ))
+                               if ( !repl_user )
 #endif
                                {
                                        replog( be, op, e->e_dn, e );
@@ -293,13 +290,13 @@ do_add( Connection *conn, Operation *op )
                }
        } else {
 #ifdef NEW_LOGGING
-            LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
-                       "do_add: conn %d  no backend support\n", conn->c_connid ));
+           LDAP_LOG(( "operation", LDAP_LEVEL_INFO,
+                      "do_add: conn %d  no backend support\n", conn->c_connid ));
 #else
-           Debug( LDAP_DEBUG_ARGS, "    do_add: no backend support\n", 0, 0, 0 );
+           Debug( LDAP_DEBUG_ARGS, "    do_add: no backend support\n", 0, 0, 0 );
 #endif
-            send_ldap_result( conn, op, rc = LDAP_UNWILLING_TO_PERFORM,
-                              NULL, "operation not supported within namingContext", NULL, NULL );
+           send_ldap_result( conn, op, rc = LDAP_UNWILLING_TO_PERFORM,
+                             NULL, "operation not supported within namingContext", NULL, NULL );
        }
 
 done: