diff options
author | J. Bruce Fields <bfields@redhat.com> | 2014-03-10 20:50:31 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2014-03-28 21:24:56 -0400 |
commit | 1bc49d83c37cfaf46be357757e592711e67f9809 (patch) | |
tree | db24c30670d00441e881656f0d3b33d545fe3b72 | |
parent | 480efaee085235bb848f1063f959bf144103c342 (diff) |
nfsd4: fix nfs4err_resource in 4.1 case
encode_getattr, for example, can return nfserr_resource to indicate it
ran out of buffer space. That's not a legal error in the 4.1 case. And
in the 4.1 case, if we ran out of buffer space, we should have exceeded
a session limit too.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r-- | fs/nfsd/nfs4xdr.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c index 94b7f19d2718..4c247c1ea5c1 100644 --- a/fs/nfsd/nfs4xdr.c +++ b/fs/nfsd/nfs4xdr.c | |||
@@ -3625,6 +3625,14 @@ nfsd4_encode_operation(struct nfsd4_compoundres *resp, struct nfsd4_op *op) | |||
3625 | /* nfsd4_check_resp_size guarantees enough room for error status */ | 3625 | /* nfsd4_check_resp_size guarantees enough room for error status */ |
3626 | if (!op->status) | 3626 | if (!op->status) |
3627 | op->status = nfsd4_check_resp_size(resp, 0); | 3627 | op->status = nfsd4_check_resp_size(resp, 0); |
3628 | if (op->status == nfserr_resource && nfsd4_has_session(&resp->cstate)) { | ||
3629 | struct nfsd4_slot *slot = resp->cstate.slot; | ||
3630 | |||
3631 | if (slot->sl_flags & NFSD4_SLOT_CACHETHIS) | ||
3632 | op->status = nfserr_rep_too_big_to_cache; | ||
3633 | else | ||
3634 | op->status = nfserr_rep_too_big; | ||
3635 | } | ||
3628 | if (so) { | 3636 | if (so) { |
3629 | so->so_replay.rp_status = op->status; | 3637 | so->so_replay.rp_status = op->status; |
3630 | so->so_replay.rp_buflen = (char *)resp->p - (char *)(statp+1); | 3638 | so->so_replay.rp_buflen = (char *)resp->p - (char *)(statp+1); |