diff options
author | Jeff Layton <jlayton@primarydata.com> | 2014-06-17 06:14:08 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2014-06-23 11:31:38 -0400 |
commit | f7ce5d284253db9760fc1c3a96b66ec2d9abf0ab (patch) | |
tree | ee42e7e0825cfbf4f24311338b7785e0326465d7 /fs/nfsd/nfs4acl.c | |
parent | 94ec938b612eb877bb6622847972dd739ef738b8 (diff) | |
download | op-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.c | 2 |
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; } |