]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
wb_sids2xids: change 'i' to 'li' in wb_sids2xids_lookupsids_done()
authorStefan Metzmacher <metze@samba.org>
Tue, 15 Sep 2020 11:58:26 +0000 (13:58 +0200)
committerAndrew Bartlett <abartlet@samba.org>
Fri, 23 Oct 2020 03:25:37 +0000 (03:25 +0000)
With all the indexes we have into various array, this makes clear
'li' is the index into the state->lookup_sids array.

This makes the following changes easier to review.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14539

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
source3/winbindd/wb_sids2xids.c

index d6655402b57c3cb496189040e073d0abaa4e5e15..f01e36e6e55a4077cf9af0e25077688b3600632b 100644 (file)
@@ -235,7 +235,7 @@ static void wb_sids2xids_lookupsids_done(struct tevent_req *subreq)
        struct lsa_RefDomainList *domains = NULL;
        struct lsa_TransNameArray *names = NULL;
        NTSTATUS status;
-       uint32_t i;
+       uint32_t li;
 
        status = wb_lookupsids_recv(subreq, state, &domains, &names);
        TALLOC_FREE(subreq);
@@ -250,11 +250,11 @@ static void wb_sids2xids_lookupsids_done(struct tevent_req *subreq)
                return;
        }
 
-       for (i=0; i<state->lookup_count; i++) {
-               const struct dom_sid *sid = &state->lookup_sids[i];
+       for (li = 0; li < state->lookup_count; li++) {
+               const struct dom_sid *sid = &state->lookup_sids[li];
                struct dom_sid dom_sid;
-               struct lsa_TranslatedName *n = &names->names[i];
-               struct wbint_TransID *t = &state->ids.ids[i];
+               struct lsa_TranslatedName *n = &names->names[li];
+               struct wbint_TransID *t = &state->ids.ids[li];
                int domain_index;
                const char *domain_name = NULL;