summaryrefslogtreecommitdiff
path: root/source3/winbindd
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2009-08-22 18:35:52 +0200
committerVolker Lendecke <vl@samba.org>2009-08-23 10:19:32 +0200
commita09b9ed9e2d4ce02dffa156049f266ba5bb59c49 (patch)
treee72d7b0cc444996830501183ed3df9dfaa93f805 /source3/winbindd
parentda9356711b14d7475bcfe4cf0bb1874c018db276 (diff)
downloadsamba-a09b9ed9e2d4ce02dffa156049f266ba5bb59c49.tar.gz
samba-a09b9ed9e2d4ce02dffa156049f266ba5bb59c49.tar.bz2
samba-a09b9ed9e2d4ce02dffa156049f266ba5bb59c49.zip
s3:winbind: Fix the talloc hierarchy in wb_queryuser_done
We need to return state->userinfo beyond the end of wb_queryuser_recv, so the unmarshalled strings are children of that, not the state that is lost sooner. Metze, this scheme works fine as long as we only have a single malloc'ed entity that is returned. I think we need a different scheme in the future when we might have more than one independent object to be returned.
Diffstat (limited to 'source3/winbindd')
-rw-r--r--source3/winbindd/wb_queryuser.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/winbindd/wb_queryuser.c b/source3/winbindd/wb_queryuser.c
index 1af44000fe..7a2f1de17b 100644
--- a/source3/winbindd/wb_queryuser.c
+++ b/source3/winbindd/wb_queryuser.c
@@ -77,7 +77,7 @@ static void wb_queryuser_done(struct tevent_req *subreq)
req, struct wb_queryuser_state);
NTSTATUS status, result;
- status = rpccli_wbint_QueryUser_recv(subreq, state, &result);
+ status = rpccli_wbint_QueryUser_recv(subreq, state->info, &result);
TALLOC_FREE(subreq);
if (!NT_STATUS_IS_OK(status)) {
tevent_req_nterror(req, status);