summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@redhat.com>2012-09-11 15:33:21 -0400
committerJ. Bruce Fields <bfields@redhat.com>2012-10-01 17:39:58 -0400
commit497305014864ea9e32adf7e8a06081209a5e7c42 (patch)
treecbbc5f14407e18bd6908cbdc2237156ca8e2cc96 /fs
parentd15c077e442d3c4167aaac87b3b7fe8ccad67a1f (diff)
downloadop-kernel-dev-497305014864ea9e32adf7e8a06081209a5e7c42.zip
op-kernel-dev-497305014864ea9e32adf7e8a06081209a5e7c42.tar.gz
nfsd4: reject bad forechannel attrs earlier
This could simplify the logic a little later. Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/nfsd/nfs4state.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 5507842..64938ca 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -1771,6 +1771,8 @@ nfsd4_create_session(struct svc_rqst *rqstp,
if (cr_ses->flags & ~SESSION4_FLAG_MASK_A)
return nfserr_inval;
+ if (check_forechannel_attrs(cr_ses->fore_channel))
+ return nfserr_toosmall;
nfs4_lock_state();
unconf = find_unconfirmed_client(&cr_ses->clientid, true);
@@ -1811,10 +1813,6 @@ nfsd4_create_session(struct svc_rqst *rqstp,
cr_ses->flags &= ~SESSION4_PERSIST;
cr_ses->flags &= ~SESSION4_RDMA;
- status = nfserr_toosmall;
- if (check_forechannel_attrs(cr_ses->fore_channel))
- goto out;
-
status = nfserr_jukebox;
new = alloc_init_session(rqstp, conf, cr_ses);
if (!new)
OpenPOWER on IntegriCloud