From: Anders Kaseorg <andersk@mit.edu>
Date: Tue, 5 Nov 2013 06:11:15 +0000 (-0500)
Subject: afs_linux_pag_from_groups: Stop checking for NOGROUP sentinel
X-Git-Tag: upstream/1.8.0_pre1^2~909
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=231e50ff9742927188d67099f0e1dbaf09858c3c;p=packages%2Fo%2Fopenafs.git

afs_linux_pag_from_groups: Stop checking for NOGROUP sentinel

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 <andersk@mit.edu>
Reviewed-on: http://gerrit.openafs.org/10426
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

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;
     }