Used API signature from back-bdb; compiles and passes make test

diff --git a/servers/slapd/back-ldbm/add.c b/servers/slapd/back-ldbm/add.c
index 65fa8748a9e2861e572e2c4fee289ec1957db4a7..5ff80d36b16f11a9331431c7971a34077cb78f43 100644 (file)
--- a/servers/slapd/back-ldbm/add.c
+++ b/servers/slapd/back-ldbm/add.c
@@ -28,7 +28,8 @@ ldbm_back_add(
        char            *pdn;
        Entry           *p = NULL;
        int                     rootlock = 0;
        char            *pdn;
        Entry           *p = NULL;
        int                     rootlock = 0;

-       int                     rc, rc_id; 
+       int                     rc;
+       ID               id = NOID;

        const char      *text = NULL;
        AttributeDescription *children = slap_schema.si_ad_children;
        char textbuf[SLAP_TEXT_BUFLEN];
        const char      *text = NULL;
        AttributeDescription *children = slap_schema.si_ad_children;
        char textbuf[SLAP_TEXT_BUFLEN];


@@ -45,11 +46,11 @@ ldbm_back_add(
        /* nobody else can add until we lock our parent */
        ldap_pvt_thread_mutex_lock(&li->li_add_mutex);
 
        /* nobody else can add until we lock our parent */
        ldap_pvt_thread_mutex_lock(&li->li_add_mutex);
 

-       if ( ( dn2id( be, e->e_ndn, &rc_id ) ) != NOID || rc_id ) {
-               /* if (rc_id) something bad happened to ldbm cache */
+       if ( ( rc = dn2id( be, e->e_ndn, &id ) ) || id != NOID ) {
+               /* if (rc) something bad happened to ldbm cache */

                ldap_pvt_thread_mutex_unlock(&li->li_add_mutex);
                send_ldap_result( conn, op, 
                ldap_pvt_thread_mutex_unlock(&li->li_add_mutex);
                send_ldap_result( conn, op, 

-                       rc_id ? LDAP_OPERATIONS_ERROR : LDAP_ALREADY_EXISTS,
+                       rc ? LDAP_OPERATIONS_ERROR : LDAP_ALREADY_EXISTS,

                        NULL, NULL, NULL, NULL );
                return( -1 );
        }
                        NULL, NULL, NULL, NULL );
                return( -1 );
        }

diff --git a/servers/slapd/back-ldbm/dn2id.c b/servers/slapd/back-ldbm/dn2id.c
index 0dc80f0710123771d2f3b21a3c93c70614e4cac9..088bc45bb033db5d4d113b7154ebada9e2978535 100644 (file)
--- a/servers/slapd/back-ldbm/dn2id.c
+++ b/servers/slapd/back-ldbm/dn2id.c
@@ -113,20 +113,17 @@ dn2id_add(
        return( rc );
 }
 
        return( rc );
 }
 

-ID
+int

 dn2id(
     Backend    *be,
     const char *dn,
 dn2id(
     Backend    *be,
     const char *dn,

-    int         *rc
+    ID          *idp

 )
 {
        struct ldbminfo *li = (struct ldbminfo *) be->be_private;
        DBCache *db;
 )
 {
        struct ldbminfo *li = (struct ldbminfo *) be->be_private;
        DBCache *db;

-       ID              id;

        Datum           key, data;
 
        Datum           key, data;
 

-       *rc = 0;
-       

 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
                   "dn2id: (%s)\n", dn ));
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
                   "dn2id: (%s)\n", dn ));


@@ -134,18 +131,19 @@ dn2id(
        Debug( LDAP_DEBUG_TRACE, "=> dn2id( \"%s\" )\n", dn, 0, 0 );
 #endif
 
        Debug( LDAP_DEBUG_TRACE, "=> dn2id( \"%s\" )\n", dn, 0, 0 );
 #endif
 

+       assert( idp );

 
        /* first check the cache */
 
        /* first check the cache */

-       if ( (id = cache_find_entry_dn2id( be, &li->li_cache, dn )) != NOID ) {
+       if ( (*idp = cache_find_entry_dn2id( be, &li->li_cache, dn )) != NOID ) {

 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_DETAIL1,
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_DETAIL1,

-                          "dn2id: (%s)%ld in cache.\n", dn, id ));
+                          "dn2id: (%s)%ld in cache.\n", dn, *idp ));

 #else
 #else

-               Debug( LDAP_DEBUG_TRACE, "<= dn2id %ld (in cache)\n", id,
+               Debug( LDAP_DEBUG_TRACE, "<= dn2id %ld (in cache)\n", *idp,

                        0, 0 );
 #endif
 
                        0, 0 );
 #endif
 

-               return( id );
+               return( 0 );

        }
 
        if ( (db = ldbm_cache_open( be, "dn2id", LDBM_SUFFIX, LDBM_WRCREAT ))
        }
 
        if ( (db = ldbm_cache_open( be, "dn2id", LDBM_SUFFIX, LDBM_WRCREAT ))


@@ -161,8 +159,8 @@ dn2id(
                 * return code !0 if ldbm cache open failed;
                 * callers should handle this
                 */
                 * return code !0 if ldbm cache open failed;
                 * callers should handle this
                 */

-               *rc = -1;
-               return( NOID );
+               *idp = NOID;
+               return( -1 );

        }
 
        ldbm_datum_init( key );
        }
 
        ldbm_datum_init( key );


@@ -185,23 +183,24 @@ dn2id(
                Debug( LDAP_DEBUG_TRACE, "<= dn2id NOID\n", 0, 0, 0 );
 #endif
 
                Debug( LDAP_DEBUG_TRACE, "<= dn2id NOID\n", 0, 0, 0 );
 #endif
 

-               return( NOID );
+               *idp = NOID;
+               return( 0 );

        }
 
        }
 

-       AC_MEMCPY( (char *) &id, data.dptr, sizeof(ID) );
+       AC_MEMCPY( (char *) idp, data.dptr, sizeof(ID) );

 
 

-       assert( id != NOID );
+       assert( *idp != NOID );

 
        ldbm_datum_free( db->dbc_db, data );
 
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
 
        ldbm_datum_free( db->dbc_db, data );
 
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,

-                  "dn2id: %ld\n", id ));
+                  "dn2id: %ld\n", *idp ));

 #else
 #else

-       Debug( LDAP_DEBUG_TRACE, "<= dn2id %ld\n", id, 0, 0 );
+       Debug( LDAP_DEBUG_TRACE, "<= dn2id %ld\n", *idp, 0, 0 );

 #endif
 
 #endif
 

-       return( id );
+       return( 0 );

 }
 
 ID_BLOCK *
 }
 
 ID_BLOCK *


@@ -363,7 +362,6 @@ dn2entry_rw(
        ID              id;
        Entry           *e = NULL;
        char            *pdn;
        ID              id;
        Entry           *e = NULL;
        char            *pdn;

-       int             rc_id = 0;

 
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
 
 #ifdef NEW_LOGGING
        LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,


@@ -380,13 +378,16 @@ dn2entry_rw(
                *matched = NULL;
        }
 
                *matched = NULL;
        }
 

-       if ( (id = dn2id( be, dn, &rc_id )) != NOID &&
-               (e = id2entry_rw( be, id, rw )) != NULL )
-       {
-               return( e );
-       }
+       if ( dn2id( be, dn, &id ) ) {
+               /* something bad happened to ldbm cache */
+               return( NULL );
+
+       } else if ( id != NOID ) {
+               /* try to return the entry */
+               if ((e = id2entry_rw( be, id, rw )) != NULL ) {
+                       return( e );
+               }

 
 

-       if ( id != NOID ) {

 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_ERR,
                           "dn2entry_rw: no entry for valid id (%ld), dn (%s)\n",
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_ERR,
                           "dn2entry_rw: no entry for valid id (%ld), dn (%s)\n",


@@ -399,9 +400,6 @@ dn2entry_rw(
 
                /* must have been deleted from underneath us */
                /* treat as if NOID was found */
 
                /* must have been deleted from underneath us */
                /* treat as if NOID was found */

-       } else if ( rc_id ) {
-               /* something bad happened to ldbm cache */
-               return NULL;

        }
 
        /* caller doesn't care about match */
        }
 
        /* caller doesn't care about match */

diff --git a/servers/slapd/back-ldbm/modrdn.c b/servers/slapd/back-ldbm/modrdn.c
index 01661991e08941160091dcb158e5982045494ff5..ed804339ad83010429371fe0701430db825a59c0 100644 (file)
--- a/servers/slapd/back-ldbm/modrdn.c
+++ b/servers/slapd/back-ldbm/modrdn.c
@@ -52,6 +52,7 @@ ldbm_back_modrdn(
        Entry           *matched;
        int                     rootlock = 0;
        int                     rc = -1, rc_id = 0;
        Entry           *matched;
        int                     rootlock = 0;
        int                     rc = -1, rc_id = 0;

+       ID              id = NOID;

        const char *text = NULL;
        char textbuf[SLAP_TEXT_BUFLEN];
        size_t textlen = sizeof textbuf;
        const char *text = NULL;
        char textbuf[SLAP_TEXT_BUFLEN];
        size_t textlen = sizeof textbuf;


@@ -380,7 +381,7 @@ ldbm_back_modrdn(
        }
 
        ldap_pvt_thread_mutex_unlock( &op->o_abandonmutex );
        }
 
        ldap_pvt_thread_mutex_unlock( &op->o_abandonmutex );

-       if (dn2id ( be, new_ndn, &rc_id ) != NOID || rc_id ) {
+       if ( ( rc_id = dn2id ( be, new_ndn, &id ) ) || id != NOID ) {

                /* if (rc_id) something bad happened to ldbm cache */
                send_ldap_result( conn, op, 
                        rc_id ? LDAP_OPERATIONS_ERROR : LDAP_ALREADY_EXISTS,
                /* if (rc_id) something bad happened to ldbm cache */
                send_ldap_result( conn, op, 
                        rc_id ? LDAP_OPERATIONS_ERROR : LDAP_ALREADY_EXISTS,

diff --git a/servers/slapd/back-ldbm/proto-back-ldbm.h b/servers/slapd/back-ldbm/proto-back-ldbm.h
index ccefe6d5830d2bbe399e4c597f4b7f783753bd05..350120a3b0195569cc7b95430511653760903759 100644 (file)
--- a/servers/slapd/back-ldbm/proto-back-ldbm.h
+++ b/servers/slapd/back-ldbm/proto-back-ldbm.h
@@ -77,7 +77,7 @@ void *ldbm_cache_sync_daemon LDAP_P(( void *));
  */
 
 int dn2id_add LDAP_P(( Backend *be, const char *dn, ID id ));
  */
 
 int dn2id_add LDAP_P(( Backend *be, const char *dn, ID id ));

-ID dn2id LDAP_P(( Backend *be, const char *dn, int *rc ));
+int dn2id LDAP_P(( Backend *be, const char *dn, ID *idp ));

 ID_BLOCK *dn2idl LDAP_P(( Backend *be, const char *dn, int prefix ));
 int dn2id_delete LDAP_P(( Backend *be, const char *dn, ID id ));
 
 ID_BLOCK *dn2idl LDAP_P(( Backend *be, const char *dn, int prefix ));
 int dn2id_delete LDAP_P(( Backend *be, const char *dn, ID id ));
 

diff --git a/servers/slapd/back-ldbm/tools.c b/servers/slapd/back-ldbm/tools.c
index 6bbbfc61669198d396e3168db847e7f87b9906e5..94d0dc6387d6cf2804a4d3f83d2db30683565865 100644 (file)
--- a/servers/slapd/back-ldbm/tools.c
+++ b/servers/slapd/back-ldbm/tools.c
@@ -156,7 +156,7 @@ ID ldbm_tool_entry_put(
 {
        struct ldbminfo *li = (struct ldbminfo *) be->be_private;
        Datum key, data;
 {
        struct ldbminfo *li = (struct ldbminfo *) be->be_private;
        Datum key, data;

-       int rc, len, rc_id;
+       int rc, len;

        ID id;
 
        assert( slapMode & SLAP_TOOL_MODE );
        ID id;
 
        assert( slapMode & SLAP_TOOL_MODE );


@@ -176,7 +176,11 @@ ID ldbm_tool_entry_put(
                e->e_id, e->e_dn, 0 );
 #endif
 
                e->e_id, e->e_dn, 0 );
 #endif
 

-       id = dn2id( be, e->e_ndn, &rc_id );
+       if ( dn2id( be, e->e_ndn, &id ) ) {
+               /* something bad happened to ldbm cache */
+               return NOID;
+       }
+

        if( id != NOID ) {
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,
        if( id != NOID ) {
 #ifdef NEW_LOGGING
                LDAP_LOG(( "backend", LDAP_LEVEL_ENTRY,


@@ -188,9 +192,6 @@ ID ldbm_tool_entry_put(
                        e->e_ndn, id, 0 );
 #endif
                return NOID;
                        e->e_ndn, id, 0 );
 #endif
                return NOID;

-       } else if ( rc_id ) {
-               /* something bad happened to ldbm cache */
-               return NOID;

        }
 
        rc = index_entry_add( be, e, e->e_attrs );
        }
 
        rc = index_entry_add( be, e, e->e_attrs );