diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2007-11-20 15:54:10 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2008-02-01 16:42:06 -0500 |
commit | 366e0c1d9116ed03320779ecf9c162204f4c712e (patch) | |
tree | 5610f88a5a98d1ba4544d7ed18a9d497f3c63228 /fs | |
parent | f3aba4e5a1b963c8bd43394cb15fb9fb6a229cd2 (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>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/nfsd/nfs4state.c | 4 |
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 |