From 231e50ff9742927188d67099f0e1dbaf09858c3c Mon Sep 17 00:00:00 2001 From: Anders Kaseorg Date: Tue, 5 Nov 2013 01:11:15 -0500 Subject: [PATCH] afs_linux_pag_from_groups: Stop checking for NOGROUP sentinel MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Linux hasn’t used NOGROUP as a sentinel like this since before kernel 2.1.12, and OpenAFS hasn’t used it on Linux since commit 109927bf6f54b58b76ac48ba41c2012c74937fed (Remove pre-Linux 2.6 support). Change-Id: I0b18de8e5d9b6cd9b20da43ed050163c2d8651ff Signed-off-by: Anders Kaseorg Reviewed-on: http://gerrit.openafs.org/10426 Tested-by: BuildBot Reviewed-by: Marc Dionne Reviewed-by: Derrick Brashear --- src/afs/LINUX/osi_groups.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/afs/LINUX/osi_groups.c b/src/afs/LINUX/osi_groups.c index 9916d2b42..88eeb6152 100644 --- a/src/afs/LINUX/osi_groups.c +++ b/src/afs/LINUX/osi_groups.c @@ -37,8 +37,8 @@ afs_linux_pag_from_groups(struct group_info *group_info) { if (group_info->ngroups < NUMPAGGROUPS) return NOPAG; - for (i = 0; (i < group_info->ngroups && - (g0 = GROUP_AT(group_info, i)) != (gid_t) NOGROUP); i++) { + for (i = 0; i < group_info->ngroups; i++) { + g0 = GROUP_AT(group_info, i); if (((g0 >> 24) & 0xff) == 'A') return g0; } -- 2.39.5