sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2024-12-23 17:34:34 +03:00
Code

winbind: Remove a level of indirection

Browse Source 
idmap_doms does not need a talloc of its own

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
This commit is contained in:
Volker Lendecke 2015-12-29 11:59:54 +01:00 committed by Ralph Boehme
parent 6018a7756f
commit 9743be68d7
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
source3/winbindd/wb_sids2xids.c
View File

@ -47,7 +47,7 @@ struct wb_sids2xids_state {
* new domain, but this approach avoids id mappings for
* invalid SIDs.
*/
struct lsa_RefDomainList *idmap_doms;
struct lsa_RefDomainList idmap_doms;
struct wbint_TransIDArray ids;
};
@ -174,11 +174,6 @@ static void wb_sids2xids_lookupsids_done(struct tevent_req *subreq)
return;
}
state->idmap_doms = talloc_zero(state, struct lsa_RefDomainList);
if (tevent_req_nomem(state->idmap_doms, req)) {
return;
}
for (i=0; i<state->num_non_cached; i++) {
struct dom_sid dom_sid;
struct lsa_DomainInfo *info;
@ -193,7 +188,7 @@ static void wb_sids2xids_lookupsids_done(struct tevent_req *subreq)
t->type = lsa_SidType_to_id_type(n->sid_type);
domain_index = init_lsa_ref_domain_list(
state, state->idmap_doms, info->name.string, &dom_sid);
state, &state->idmap_doms, info->name.string, &dom_sid);
if (domain_index == -1) {
tevent_req_oom(req);
return;
@ -210,7 +205,7 @@ static void wb_sids2xids_lookupsids_done(struct tevent_req *subreq)
child = idmap_child();
subreq = dcerpc_wbint_Sids2UnixIDs_send(
state, state->ev, child->binding_handle, state->idmap_doms,
state, state->ev, child->binding_handle, &state->idmap_doms,
&state->ids);
if (tevent_req_nomem(subreq, req)) {
return;