diff options
author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2011-12-22 16:49:48 +0100 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2011-12-23 10:36:19 +0100 |
commit | 2bc7b8f4c9992f5f825d8328e0353f793d7e7ebf (patch) | |
tree | f017233c82f8f66932aaf37456c0497f89622c4c /source4/libcli | |
parent | eddf0d0fe277cc763bbe0bb4706ab1a4aebf8151 (diff) | |
download | samba-2bc7b8f4c9992f5f825d8328e0353f793d7e7ebf.tar.gz samba-2bc7b8f4c9992f5f825d8328e0353f793d7e7ebf.tar.bz2 samba-2bc7b8f4c9992f5f825d8328e0353f793d7e7ebf.zip |
s4:libcli/finddcs_nbt.c - free "req" consistently with "finddcs_cldap.c"
It is more obvious to free where the context for the first time appears.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source4/libcli')
-rw-r--r-- | source4/libcli/finddcs_nbt.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source4/libcli/finddcs_nbt.c b/source4/libcli/finddcs_nbt.c index 9579a9e449..6a5d1fd856 100644 --- a/source4/libcli/finddcs_nbt.c +++ b/source4/libcli/finddcs_nbt.c @@ -284,7 +284,6 @@ NTSTATUS finddcs_nbt_recv(struct tevent_req *req, TALLOC_CTX *mem_ctx, *num_dcs = state->num_dcs; *dcs = talloc_steal(mem_ctx, state->dcs); } - talloc_free(req); return status; } @@ -298,6 +297,7 @@ NTSTATUS finddcs_nbt(TALLOC_CTX *mem_ctx, struct imessaging_context *msg_ctx, int *num_dcs, struct nbt_dc_name **dcs) { + NTSTATUS status; struct tevent_req *req = finddcs_nbt_send(mem_ctx, my_netbios_name, nbt_port, @@ -305,5 +305,7 @@ NTSTATUS finddcs_nbt(TALLOC_CTX *mem_ctx, domain_sid, resolve_ctx, event_ctx, msg_ctx); - return finddcs_nbt_recv(req, mem_ctx, num_dcs, dcs); + status = finddcs_nbt_recv(req, mem_ctx, num_dcs, dcs); + talloc_free(req); + return status; } |