diff options
| author | Volker Lendecke <vl@samba.org> | 2009-08-22 18:35:52 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2009-08-23 10:19:32 +0200 |
| commit | a09b9ed9e2d4ce02dffa156049f266ba5bb59c49 (patch) | |
| tree | e72d7b0cc444996830501183ed3df9dfaa93f805 /source3 | |
| parent | da9356711b14d7475bcfe4cf0bb1874c018db276 (diff) | |
| download | samba-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')
| -rw-r--r-- | source3/winbindd/wb_queryuser.c | 2 |
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); |