summaryrefslogtreecommitdiffstats
path: root/fs/nfsd/nfs4acl.c
diff options
context:
space:
mode:
authorJeff Layton <jlayton@primarydata.com>2014-06-17 06:14:08 -0400
committerJ. Bruce Fields <bfields@redhat.com>2014-06-23 11:31:38 -0400
commitf7ce5d284253db9760fc1c3a96b66ec2d9abf0ab (patch)
treeee42e7e0825cfbf4f24311338b7785e0326465d7 /fs/nfsd/nfs4acl.c
parent94ec938b612eb877bb6622847972dd739ef738b8 (diff)
downloadop-kernel-dev-f7ce5d284253db9760fc1c3a96b66ec2d9abf0ab.zip
op-kernel-dev-f7ce5d284253db9760fc1c3a96b66ec2d9abf0ab.tar.gz
nfsd: fix return of nfs4_acl_write_who
AFAICT, the only way to hit this error is to pass this function a bogus "who" value. In that case, we probably don't want to return -1 as that could get sent back to the client. Turn this into nfserr_serverfault, which is a more appropriate error for a server bug like this. Signed-off-by: Jeff Layton <jlayton@primarydata.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4acl.c')
-rw-r--r--fs/nfsd/nfs4acl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4acl.c b/fs/nfsd/nfs4acl.c
index d714156..b0cf00d 100644
--- a/fs/nfsd/nfs4acl.c
+++ b/fs/nfsd/nfs4acl.c
@@ -935,5 +935,5 @@ __be32 nfs4_acl_write_who(struct xdr_stream *xdr, int who)
return 0;
}
WARN_ON_ONCE(1);
- return -1;
+ return nfserr_serverfault;
}
OpenPOWER on IntegriCloud