diff options
| author | Nalin Dahyabhai <nalin.dahyabhai@pobox.com> | 2008-11-19 17:20:41 -0500 |
|---|---|---|
| committer | Nalin Dahyabhai <nalin.dahyabhai@pobox.com> | 2008-11-19 17:20:41 -0500 |
| commit | e705dbadd38d55090e2764f0fe656c948a92c14b (patch) | |
| tree | d0c485639c8f2d525c6b7c55312f2c1b7ca7f1c7 | |
| parent | c2e0b2669f939bc1a5c3fc3844fb110c4276b310 (diff) | |
- remove a redundant read lock in the schema-compat plugin, call it 0.8.2slapi-nis-0.8.2
| -rw-r--r-- | configure.ac | 2 | ||||
| -rw-r--r-- | slapi-nis.spec | 5 | ||||
| -rw-r--r-- | src/back-sch.c | 19 |
3 files changed, 8 insertions, 18 deletions
diff --git a/configure.ac b/configure.ac index 91bcb47..c76b4fc 100644 --- a/configure.ac +++ b/configure.ac @@ -1,4 +1,4 @@ -AC_INIT(slapi-nis,0.8.1) +AC_INIT(slapi-nis,0.8.2) AM_INIT_AUTOMAKE(foreign) AC_PROG_LIBTOOL AM_CONDITIONAL(GCC,[test x$GCC = xyes]) diff --git a/slapi-nis.spec b/slapi-nis.spec index e1824ea..67bea7a 100644 --- a/slapi-nis.spec +++ b/slapi-nis.spec @@ -1,5 +1,5 @@ Name: slapi-nis -Version: 0.8.1 +Version: 0.8.2 Release: 1%{?dist} Summary: NIS Server and Schema Compatibility plugins for Directory Server Group: System Environment/Daemons @@ -46,6 +46,9 @@ rm -rf $RPM_BUILD_ROOT %{_sbindir}/nisserver-plugin-defs %changelog +* Wed Nov 19 2008 Nalin Dahyabhai <nalin@redhat.com> - 0.8.2-1 +- update to 0.8.2 to remove a redundant read lock in the schema-compat plugin + * Fri Oct 3 2008 Nalin Dahyabhai <nalin@redhat.com> - 0.8.1-1 - update to 0.8.1 to fix a heap corruption (Rich Megginson) diff --git a/src/back-sch.c b/src/back-sch.c index 49e1dd5..07a3fd0 100644 --- a/src/back-sch.c +++ b/src/back-sch.c @@ -909,33 +909,20 @@ backend_group_check_scope_cb(const char *group, void *cb_data) } static bool_t -backend_check_scope(struct plugin_state *state, const char *target) +backend_check_scope_pb(Slapi_PBlock *pb) { struct backend_group_check_scope_cbdata cbdata; - cbdata.state = state; - cbdata.target = target; + slapi_pblock_get(pb, SLAPI_PLUGIN_PRIVATE, &cbdata.state); + 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; } -static bool_t -backend_check_scope_pb(Slapi_PBlock *pb) -{ - struct plugin_state *state; - char *target; - - slapi_pblock_get(pb, SLAPI_PLUGIN_PRIVATE, &state); - slapi_pblock_get(pb, SLAPI_TARGET_DN, &target); - return backend_check_scope(state, target); -} - static int backend_write_cb(Slapi_PBlock *pb) { |