Multiple group semantics broken in 2.1.31 + patch

James Bottomley (James.Bottomley@columbiasc.ncr.com)
Fri, 04 Apr 1997 10:34:22 -0600

This is a multipart MIME message.

--===_0_Fri_Apr__4_10:29:16_CST_1997
Content-Type: text/plain; charset=us-ascii

2.1.31 will only give you access to files based on the first group in your
current->groups list rather than checking all of them. Attached is a patch to
correct this. For some reason this behaviour was explicitly introduced in
2.1.31, is there some problem I'm missing?

James Bottomley

--===_0_Fri_Apr__4_10:29:16_CST_1997
Content-Type: application/x-patch
Content-Description: groups.diff

--- 1.1 1997/04/04 16:17:48
+++ kernel/sys.c 1997/04/04 16:18:02
@@ -893,7 +893,7 @@
gid_t *groups = current->groups;
do {
if (*groups == grp)
- break;
+ return 1;
groups++;
i--;
} while (i);

--===_0_Fri_Apr__4_10:29:16_CST_1997--