upstream/openldap
1
0
Fork 0
mirror of https://git.openldap.org/openldap/openldap.git synced 2025-12-24 00:29:35 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

ITS#6760: Fix rwm entry handling

Browse source
This commit is contained in:
Hallvard Furuseth 2010-12-31 12:01:03 +00:00
parent 13f52646b2
commit 04f8fa5619
1 changed files with 26 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

45
servers/slapd/overlays/rwm.c
View file

@ -911,9 +911,16 @@ rwm_entry_get_rw( Operation *op, struct berval *ndn,
/* duplicate & release */
op2.o_bd->bd_info = (BackendInfo *)on;
rc = rwm_send_entry( &op2, &rs );
RS_ASSERT( rs.sr_flags & REP_ENTRY_MUSTFLUSH );
if ( rc == SLAP_CB_CONTINUE ) {
*ep = rs.sr_entry;
rc = LDAP_SUCCESS;
} else {
assert( rc != LDAP_SUCCESS && rs.sr_entry == *ep );
*ep = NULL;
op2.o_bd->bd_info = (BackendInfo *)on->on_info;
be_entry_release_r( &op2, rs.sr_entry );
op2.o_bd->bd_info = (BackendInfo *)on;
}
}
@ -1456,6 +1463,7 @@ cleanup_attr:;
return 0;
}
/* Should return SLAP_CB_CONTINUE or failure, never LDAP_SUCCESS. */
static int
rwm_send_entry( Operation *op, SlapReply *rs )
{
@ -1464,7 +1472,6 @@ rwm_send_entry( Operation *op, SlapReply *rs )
(struct ldaprwmap *)on->on_bi.bi_private;
Entry *e = NULL;
slap_mask_t flags;
struct berval dn = BER_BVNULL,
ndn = BER_BVNULL;
dncookie dc;
@ -1481,7 +1488,6 @@ rwm_send_entry( Operation *op, SlapReply *rs )
dc.ctx = "searchEntryDN";
e = rs->sr_entry;
flags = rs->sr_flags;
if ( !( rs->sr_flags & REP_ENTRY_MODIFIABLE ) ) {
/* FIXME: all we need to duplicate are:
* - dn
@ -1489,15 +1495,17 @@ rwm_send_entry( Operation *op, SlapReply *rs )
* - attributes that are requested
* - no values if attrsonly is set
*/
e = entry_dup( e );
if ( e == NULL ) {
rc = LDAP_NO_MEMORY;
goto fail;
}
flags &= ~REP_ENTRY_MUSTRELEASE;
flags |= ( REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED );
} else if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
/* ITS#6423: REP_ENTRY_MUSTRELEASE incompatible
* with REP_ENTRY_MODIFIABLE */
RS_ASSERT( 0 );
rc = 1;
goto fail;
}
/*
@ -1529,22 +1537,21 @@ rwm_send_entry( Operation *op, SlapReply *rs )
* to return, and remap them accordingly */
(void)rwm_attrs( op, rs, &e->e_attrs, 1 );
if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
/* ITS#6423: REP_ENTRY_MUSTRELEASE incompatible
* with REP_ENTRY_MODIFIABLE */
if ( rs->sr_entry == e ) {
rc = 1;
goto fail;
if ( e != rs->sr_entry ) {
/* Reimplementing rs_replace_entry(), I suppose to
* bypass our own dubious rwm_entry_release_rw() */
if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
rs->sr_flags ^= REP_ENTRY_MUSTRELEASE;
op->o_bd->bd_info = (BackendInfo *)on->on_info;
be_entry_release_r( op, rs->sr_entry );
op->o_bd->bd_info = (BackendInfo *)on;
} else if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
entry_free( rs->sr_entry );
}
op->o_bd->bd_info = (BackendInfo *)on->on_info;
be_entry_release_r( op, rs->sr_entry );
op->o_bd->bd_info = (BackendInfo *)on;
rs->sr_entry = e;
rs->sr_flags |= REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED;
}
rs->sr_entry = e;
rs->sr_flags = flags;
return SLAP_CB_CONTINUE;
fail:;