- avoid getting read locks when we know we already have them

1 files changed, 0 insertions, 6 deletions

diff --git a/src/back-sch.c b/src/back-sch.c
index a1db2c7..aff5148 100644
--- a/src/back-sch.c
+++ b/src/back-sch.c
@@ -673,11 +673,9 @@ backend_search_set_cb(const char *group, const char *set, bool_t flag,
 		slapi_entry_add_string(set_entry,
 				       "objectClass", "extensibleObject");
 		slapi_entry_set_sdn(set_entry, set_data->container_sdn);
-		map_rdlock();
 		n_entries = map_data_get_map_size(cbdata->state,
 						  set_data->common.group,
 						  set_data->common.set);
-		map_unlock();
 		backend_set_operational_attributes(set_entry,
 						   cbdata->state, time(NULL),
 						   n_entries);
@@ -749,9 +747,7 @@ backend_search_group_cb(const char *group, void *cb_data)
 		slapi_entry_add_string(group_entry,
 				       "objectClass", "extensibleObject");
 		slapi_entry_set_sdn(group_entry, group_dn);
-		map_rdlock();
 		n_maps = map_data_get_domain_size(cbdata->state, group);
-		map_unlock();
 		backend_set_operational_attributes(group_entry, cbdata->state,
 						   time(NULL), n_maps);
 		if (!slapi_entry_rdn_values_present(group_entry)) {
@@ -983,10 +979,8 @@ backend_check_scope_pb(Slapi_PBlock *pb)
 	slapi_pblock_get(pb, SLAPI_TARGET_DN, &cbdata.target);
 	cbdata.target_dn = slapi_sdn_new_dn_byval(cbdata.target);
 	cbdata.ours = FALSE;
-	map_rdlock();
 	map_data_foreach_domain(cbdata.state, backend_group_check_scope_cb,
 				&cbdata);
-	map_unlock();
 	slapi_sdn_free(&cbdata.target_dn);
 	return cbdata.ours;
 }