don't short circuit callback list

free self in case anything goes wrong

diff --git a/contrib/slapd-modules/cloak/cloak.c b/contrib/slapd-modules/cloak/cloak.c
index 5e4f11d0e5be403f390a2111da77d2d65cdfa9bc..8d29a1579cdd78410c7ce7dc3765ad903f7e560e 100644 (file)
--- a/contrib/slapd-modules/cloak/cloak.c
+++ b/contrib/slapd-modules/cloak/cloak.c
@@ -269,8 +269,8 @@ cloak_search( Operation *op, SlapReply *rs )
 
        sc = op->o_tmpcalloc( 1, sizeof( *sc ), op->o_tmpmemctx );
        sc->sc_response = cloak_search_cb;
-       sc->sc_cleanup = NULL;
-       sc->sc_next = NULL;
+       sc->sc_cleanup = slap_freeself_cb;
+       sc->sc_next = op->o_callback;
        sc->sc_private = ci;
        op->o_callback = sc;