s3:mdssvc: add missing mds_ctx deallocation

The mds_ctx object was created in _mdssvc_open() as a talloc child of the pipe
which means as long as the pipe is connected it's not freed.

To ensure we do proper rundown of all resources including backend connections
and pending queries, we must free the mds_ctx object.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Noel Power <noel.power@suse.com>

diff --git a/source3/rpc_server/mdssvc/srv_mdssvc_nt.c b/source3/rpc_server/mdssvc/srv_mdssvc_nt.c
index c5e5a46c01e89de6e4f475a1aabaac9206f3afa0..aea6019e8717bffe881347c7b80c2973b747e1aa 100644 (file)
--- a/source3/rpc_server/mdssvc/srv_mdssvc_nt.c
+++ b/source3/rpc_server/mdssvc/srv_mdssvc_nt.c
@@ -318,6 +318,7 @@ void _mdssvc_close(struct pipes_struct *p, struct mdssvc_close *r)
        }
 
        DBG_DEBUG("Close mdssvc handle for path: %s\n", mds_ctx->spath);
+       TALLOC_FREE(mds_ctx);
 
        *r->out.out_handle = r->in.in_handle;
        close_policy_hnd(p, &r->in.in_handle);