[BK PATCH 3/4] remove NGROUPS hard limit (resend)

From: Timothy Hockin (th122948@scl2.sfbay.sun.com)
Date: Tue Nov 05 2002 - 15:02:07 EST


# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
# ChangeSet 1.931 -> 1.932
# include/linux/sunrpc/svcauth.h 1.4 -> 1.5
# net/sunrpc/svcauth_unix.c 1.9 -> 1.10
# fs/nfsd/auth.c 1.1 -> 1.2
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 02/11/05 thockin@freakshow.cobalt.com 1.932
# fix usage of NGROUPS in nfsd and svcauth
# --------------------------------------------
#
diff -Nru a/fs/nfsd/auth.c b/fs/nfsd/auth.c
--- a/fs/nfsd/auth.c Tue Nov 5 11:15:32 2002
+++ b/fs/nfsd/auth.c Tue Nov 5 11:15:32 2002
@@ -10,12 +10,15 @@
 #include <linux/sunrpc/svcauth.h>
 #include <linux/nfsd/nfsd.h>
 
+extern asmlinkage long sys_setgroups(int gidsetsize, gid_t *grouplist);
+
 #define CAP_NFSD_MASK (CAP_FS_MASK|CAP_TO_MASK(CAP_SYS_RESOURCE))
 void
 nfsd_setuser(struct svc_rqst *rqstp, struct svc_export *exp)
 {
         struct svc_cred *cred = &rqstp->rq_cred;
         int i;
+ gid_t groups[SVC_CRED_NGROUPS];
 
         if (rqstp->rq_userset)
                 return;
@@ -29,7 +32,7 @@
                         cred->cr_uid = exp->ex_anon_uid;
                 if (!cred->cr_gid)
                         cred->cr_gid = exp->ex_anon_gid;
- for (i = 0; i < NGROUPS; i++)
+ for (i = 0; i < SVC_CRED_NGROUPS; i++)
                         if (!cred->cr_groups[i])
                                 cred->cr_groups[i] = exp->ex_anon_gid;
         }
@@ -42,13 +45,13 @@
                 current->fsgid = cred->cr_gid;
         else
                 current->fsgid = exp->ex_anon_gid;
- for (i = 0; i < NGROUPS; i++) {
+ for (i = 0; i < SVC_CRED_NGROUPS; i++) {
                 gid_t group = cred->cr_groups[i];
                 if (group == (gid_t) NOGROUP)
                         break;
- current->groups[i] = group;
+ groups[i] = group;
         }
- current->ngroups = i;
+ sys_setgroups(i, groups);
 
         if ((cred->cr_uid)) {
                 cap_t(current->cap_effective) &= ~CAP_NFSD_MASK;
diff -Nru a/include/linux/sunrpc/svcauth.h b/include/linux/sunrpc/svcauth.h
--- a/include/linux/sunrpc/svcauth.h Tue Nov 5 11:15:32 2002
+++ b/include/linux/sunrpc/svcauth.h Tue Nov 5 11:15:32 2002
@@ -14,10 +14,11 @@
 #include <linux/sunrpc/msg_prot.h>
 #include <linux/sunrpc/cache.h>
 
+#define SVC_CRED_NGROUPS 32
 struct svc_cred {
         uid_t cr_uid;
         gid_t cr_gid;
- gid_t cr_groups[NGROUPS];
+ gid_t cr_groups[SVC_CRED_NGROUPS];
 };
 
 struct svc_rqst; /* forward decl */
diff -Nru a/net/sunrpc/svcauth_unix.c b/net/sunrpc/svcauth_unix.c
--- a/net/sunrpc/svcauth_unix.c Tue Nov 5 11:15:32 2002
+++ b/net/sunrpc/svcauth_unix.c Tue Nov 5 11:15:32 2002
@@ -401,11 +401,11 @@
         if (slen > 16 || (len -= (slen + 2)*4) < 0)
                 goto badcred;
         for (i = 0; i < slen; i++)
- if (i < NGROUPS)
+ if (i < SVC_CRED_NGROUPS)
                         cred->cr_groups[i] = ntohl(svc_getu32(argv));
                 else
                         svc_getu32(argv);
- if (i < NGROUPS)
+ if (i < SVC_CRED_NGROUPS)
                 cred->cr_groups[i] = NOGROUP;
 
         if (svc_getu32(argv) != RPC_AUTH_NULL || svc_getu32(argv) != 0) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Nov 07 2002 - 22:00:39 EST