commit 9831975c767bce45769ad42d916991d2f4caa1dd
author Bhavna Sharma <sbhavna@codeaurora.org> Mon Aug 27 15:32:35 2018 -0700
committer Bhavna Sharma <sbhavna@codeaurora.org> Tue Aug 28 13:47:03 2018 -0700
tree 00a893e43ffbbe85cbb7847de70d1dd0a66cda71
parent a190b2c1164a60a2350b5ed53fa4ef3e2d7e7689

Fix issue with use of incorrect index in group list

If a group is not found there is a mismatch with group count
and actual number of groups in the list leading to some
groups not getting set correctly.

CRs-Fixed: 2303952

Change-Id: Iafb3aa4e90a79b511f3d36e42583e013fd05391b

utils/loc_cfg.cpp

diff --git a/utils/loc_cfg.cpp b/utils/loc_cfg.cpp
index 4c3bc58..d319fb4 100644
--- a/utils/loc_cfg.cpp
+++ b/utils/loc_cfg.cpp
@@ -835,11 +835,9 @@
         for(i=0; i<ngroups; i++) {
             struct group* grp = getgrnam(split_strings[i]);
             if (grp) {
-                child_proc[j].group_list[i] = grp->gr_gid;
+                child_proc[j].group_list[child_proc[j].num_groups] = grp->gr_gid;
                 child_proc[j].num_groups++;
-                LOC_LOGD("%s:%d]:Group %s = %d matches child_group: %d\n",
-                         __func__, __LINE__, split_strings[i],
-                         grp->gr_gid,child_proc[j].group_list[i]);
+                LOC_LOGd("Group %s = %d", split_strings[i], grp->gr_gid);
             }
         }