[PATCH] kNFSd: Use only the uid when deciding whether a setclientid is being done...

From: NeilBrown <neilb@cse.unsw.edu.au>

I would have assumed that we should also check for the same security
(pseudo)flavor, but that doesn't seem to be how Solaris does it, and since
the spec doesn't suggest including such information in the clientid, that
would make it impossible to switch pseudoflavors.

diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 70c2b1a9e4e5050ba18c34f9a1b4c1e6c1808073..38f544c65c9027ad9c425d07a744e520365c63da 100644 (file)
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -270,8 +270,7 @@ cmp_clid(clientid_t * cl1, clientid_t * cl2) {
 /* XXX what about NGROUP */
 static int
 cmp_creds(struct svc_cred *cr1, struct svc_cred *cr2){
-       return((cr1->cr_uid == cr2->cr_uid) &&
-               (cr1->cr_gid == cr2->cr_gid));
+       return(cr1->cr_uid == cr2->cr_uid);
 
 }