library: tweak that change to 'supgrps_from_supgids()'
It's good that those Qualys folks were looking over my
shoulder. They suggested a change to that commit shown
below. This improvement was obviously a better choice.
Reference(s):
. original change
commit f9a8009e27
Signed-off-by: Jim Warner <james.warner@comcast.net>
This commit is contained in:
parent
f9a8009e27
commit
3352bddba5
@ -475,11 +475,7 @@ static void supgrps_from_supgids (proc_t *p) {
|
|||||||
|
|
||||||
while (',' == *s) ++s;
|
while (',' == *s) ++s;
|
||||||
gid = strtol(s, &end, 10);
|
gid = strtol(s, &end, 10);
|
||||||
if (end <= s) {
|
if (end <= s) break;
|
||||||
if (!p->supgrp)
|
|
||||||
p->supgrp = xstrdup("-");
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
s = end;
|
s = end;
|
||||||
g = pwcache_get_group(gid);
|
g = pwcache_get_group(gid);
|
||||||
|
|
||||||
@ -491,6 +487,9 @@ static void supgrps_from_supgids (proc_t *p) {
|
|||||||
else if (len >= max) len = max-1;
|
else if (len >= max) len = max-1;
|
||||||
t += len;
|
t += len;
|
||||||
} while (*s);
|
} while (*s);
|
||||||
|
|
||||||
|
if (!p->supgrp)
|
||||||
|
p->supgrp = xstrdup("-");
|
||||||
}
|
}
|
||||||
|
|
||||||
///////////////////////////////////////////////////////////////////////
|
///////////////////////////////////////////////////////////////////////
|
||||||
|
Loading…
Reference in New Issue
Block a user