aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@citi.umich.edu>2007-11-20 15:54:10 -0500
committerJ. Bruce Fields <bfields@citi.umich.edu>2008-02-01 16:42:06 -0500
commit366e0c1d9116ed03320779ecf9c162204f4c712e (patch)
tree5610f88a5a98d1ba4544d7ed18a9d497f3c63228
parentf3aba4e5a1b963c8bd43394cb15fb9fb6a229cd2 (diff)
nfsd4: kill unneeded cl_confirm check
We generate a unique cl_confirm for every new client; so if we've already checked that this cl_confirm agrees with the cl_confirm of unconf, then we already know that it does not agree with the cl_confirm of conf. Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
-rw-r--r--fs/nfsd/nfs4state.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 23b5fc71f9fb..60cc937b7076 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -835,9 +835,7 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp,
835 * SETCLIENTID_CONFIRM request processing consisting 835 * SETCLIENTID_CONFIRM request processing consisting
836 * of 4 bullet points, labeled as CASE1 - CASE4 below. 836 * of 4 bullet points, labeled as CASE1 - CASE4 below.
837 */ 837 */
838 if ((conf && unconf) && 838 if (conf && unconf && same_verf(&confirm, &unconf->cl_confirm)) {
839 (same_verf(&unconf->cl_confirm, &confirm)) &&
840 (!same_verf(&conf->cl_confirm, &unconf->cl_confirm))) {
841 /* 839 /*
842 * RFC 3530 14.2.34 CASE 1: 840 * RFC 3530 14.2.34 CASE 1:
843 * callback update 841 * callback update