From: Jeff Layton <jlayton@primarydata.com>
Date: Tue, 17 Jun 2014 10:14:08 +0000 (-0400)
Subject: nfsd: fix return of nfs4_acl_write_who
X-Git-Tag: v5.15~17430^2~158
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f7ce5d284253db9760fc1c3a96b66ec2d9abf0ab;p=platform%2Fkernel%2Flinux-starfive.git

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>
---

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;
 }