From 7fe9a3255ae84ed531d0a3d0a8cbf0812559fc5e Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Mon, 7 May 2007 15:27:50 +0000 Subject: r22749: fix memory leak of nbt_name_request structure which are used to send replies and never have an async callback that could free it. we only had the memory leak in the error path the standard path was ok. metze (This used to be commit d2a2fe662db5b773f4bd54498d6b31b773633903) --- source4/libcli/nbt/nbtsocket.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'source4/libcli') diff --git a/source4/libcli/nbt/nbtsocket.c b/source4/libcli/nbt/nbtsocket.c index c1e30fc245..0931de63c8 100644 --- a/source4/libcli/nbt/nbtsocket.c +++ b/source4/libcli/nbt/nbtsocket.c @@ -45,6 +45,7 @@ static int nbt_name_request_destructor(struct nbt_name_request *req) req->name_trn_id = 0; } if (req->te) { + talloc_free(req->te); req->te = NULL; } if (req->nbtsock->send_queue == NULL) { @@ -102,6 +103,8 @@ failed: talloc_free(tmp_ctx); if (req->async.fn) { req->async.fn(req); + } else if (req->is_reply) { + talloc_free(req); } return; } @@ -140,6 +143,8 @@ static void nbt_name_socket_timeout(struct event_context *ev, struct timed_event } if (req->async.fn) { req->async.fn(req); + } else if (req->is_reply) { + talloc_free(req); } } @@ -462,9 +467,7 @@ NTSTATUS nbt_name_request_recv(struct nbt_name_request *req) if (event_loop_once(req->nbtsock->event_ctx) != 0) { req->state = NBT_REQUEST_ERROR; req->status = NT_STATUS_UNEXPECTED_NETWORK_ERROR; - if (req->async.fn) { - req->async.fn(req); - } + break; } } return req->status; -- cgit