Fixed invalid cookie in pagedResults control (ITS#3089)

[openldap] / servers / slapd / back-bdb / dn2entry.c

diff --git a/servers/slapd/back-bdb/dn2entry.c b/servers/slapd/back-bdb/dn2entry.c
index b95ae4ea6a9bf80b19d636fc213b4d5b48fe6f97..25d79e5256d97ce2142db101b2d9dbe34235b48a 100644 (file)
--- a/servers/slapd/back-bdb/dn2entry.c
+++ b/servers/slapd/back-bdb/dn2entry.c
@@ -1,8 +1,17 @@
 /* dn2entry.c - routines to deal with the dn2id / id2entry glue */
 /* $OpenLDAP$ */
-/*
- * Copyright 1998-2000 The OpenLDAP Foundation, All Rights Reserved.
- * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
+/* This work is part of OpenLDAP Software <http://www.openldap.org/>.
+ *
+ * Copyright 2000-2004 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
+ * <http://www.OpenLDAP.org/license.html>.
  */
 
 #include "portable.h"
@@ -14,42 +23,59 @@
 
 /*
  * dn2entry - look up dn in the cache/indexes and return the corresponding
- * entry.
+ * entry. If the requested DN is not found and matched is TRUE, return info
+ * for the closest ancestor of the DN. Otherwise e is NULL.
  */
 
 int
 bdb_dn2entry(
-       BackendDB       *be,
+       Operation *op,
        DB_TXN *tid,
        struct berval *dn,
-       Entry **e,
-       Entry **matched,
-       int flags )
+       EntryInfo **e,
+       int matched,
+       u_int32_t locker,
+       DB_LOCK *lock )
 {
+       EntryInfo *ei = NULL;
        int rc;
-       ID              id;
-       char    *matchedDN = NULL;
 
+#ifdef NEW_LOGGING
+       LDAP_LOG ( CACHE, ARGS, "bdb_dn2entry(\"%s\")\n", dn->bv_val, 0, 0 );
+#else
        Debug(LDAP_DEBUG_TRACE, "bdb_dn2entry(\"%s\")\n",
                dn->bv_val, 0, 0 );
+#endif
 
        *e = NULL;
 
-       if( matched != NULL ) {
-               *matched = NULL;
-               rc = bdb_dn2id_matched( be, tid, dn, &id, &matchedDN );
-       } else {
-               rc = bdb_dn2id( be, tid, dn, &id );
-       }
-
-       if( rc != 0 ) {
-               return rc;
-       }
-
-       if( matchedDN == NULL ) {
-               rc = bdb_id2entry( be, tid, id, e );
+       rc = bdb_cache_find_ndn( op, tid, dn, &ei );
+       if ( rc ) {
+               if ( matched && rc == DB_NOTFOUND ) {
+                       /* Set the return value, whether we have its entry
+                        * or not.
+                        */
+                       *e = ei;
+                       if ( ei && ei->bei_id )
+                               bdb_cache_find_id( op, tid, ei->bei_id,
+                                       &ei, 1, locker, lock );
+                       else if ( ei )
+                               bdb_cache_entryinfo_unlock( ei );
+               } else if ( ei ) {
+                       bdb_cache_entryinfo_unlock( ei );
+               }
        } else {
-               rc = bdb_id2entry( be, tid, id, matched );
+               rc = bdb_cache_find_id( op, tid, ei->bei_id, &ei, 1,
+                       locker, lock );
+               if ( rc == 0 ) {
+                       *e = ei;
+               } else if ( matched && rc == DB_NOTFOUND ) {
+                       /* always return EntryInfo */
+                       ei = ei->bei_parent;
+                       bdb_cache_find_id( op, tid, ei->bei_id, &ei, 1,
+                               locker, lock );
+                       *e = ei;
+               }
        }
 
        return rc;