diff options
| author | Volker Lendecke <vl@samba.org> | 2015-03-05 20:59:16 +0100 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2015-03-07 15:28:59 +0100 |
| commit | 5ba377f3df61647e259d40a6fa24dc8445618cda (patch) | |
| tree | 502687a456fd99015041a1225665035db6a85536 /source3/winbindd/winbindd_getgroups.c | |
| parent | 91ff65b2f63061fa66b6bcc3c142c17b03581759 (diff) | |
| download | samba-5ba377f3df61647e259d40a6fa24dc8445618cda.tar.gz samba-5ba377f3df61647e259d40a6fa24dc8445618cda.tar.xz samba-5ba377f3df61647e259d40a6fa24dc8445618cda.zip | |
winbind: Make wb_sids2xids_recv work on an array
The trigger for this is that Coverity got confused by the dual use of &xid
as an array with the implicit length equality between wb_sids2xids_send
and the array passed in to wb_sids2xids_recv for the result.
I don't want to start doing things just for the Coverity scan, but this
makes the code clearer to me by removing this implicit expected array
length equality.
Signed-off-by: Volker Lendecke <vl@samba.org>
Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Sat Mar 7 15:28:59 CET 2015 on sn-devel-104
Diffstat (limited to 'source3/winbindd/winbindd_getgroups.c')
| -rw-r--r-- | source3/winbindd/winbindd_getgroups.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/winbindd/winbindd_getgroups.c b/source3/winbindd/winbindd_getgroups.c index b899bebe17..8b9d0a3ecd 100644 --- a/source3/winbindd/winbindd_getgroups.c +++ b/source3/winbindd/winbindd_getgroups.c @@ -155,7 +155,7 @@ static void winbindd_getgroups_sid2gid_done(struct tevent_req *subreq) xids[i].id = UINT32_MAX; } - status = wb_sids2xids_recv(subreq, xids); + status = wb_sids2xids_recv(subreq, xids, state->num_sids); TALLOC_FREE(subreq); if (NT_STATUS_EQUAL(status, NT_STATUS_NONE_MAPPED) || NT_STATUS_EQUAL(status, STATUS_SOME_UNMAPPED)) |