From a09b9ed9e2d4ce02dffa156049f266ba5bb59c49 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Sat, 22 Aug 2009 18:35:52 +0200 Subject: 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. --- source3/winbindd/wb_queryuser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source3/winbindd') 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); -- cgit