diff options
| author | Chuck Lever | 2016-06-29 19:55:22 +0200 |
|---|---|---|
| committer | Anna Schumaker | 2016-07-11 21:50:43 +0200 |
| commit | a4e187d83d88eeaba6252aac0a2ffe5eaa73a818 (patch) | |
| tree | 31ac04d7813866463fccf2ec9ec4dea9c88edc75 /net | |
| parent | svc: Avoid garbage replies when pc_func() returns rpc_drop_reply (diff) | |
| download | kernel-qcow2-linux-a4e187d83d88eeaba6252aac0a2ffe5eaa73a818.tar.gz kernel-qcow2-linux-a4e187d83d88eeaba6252aac0a2ffe5eaa73a818.tar.xz kernel-qcow2-linux-a4e187d83d88eeaba6252aac0a2ffe5eaa73a818.zip | |
NFS: Don't drop CB requests with invalid principals
Before commit 778be232a207 ("NFS do not find client in NFSv4
pg_authenticate"), the Linux callback server replied with
RPC_AUTH_ERROR / RPC_AUTH_BADCRED, instead of dropping the CB
request. Let's restore that behavior so the server has a chance to
do something useful about it, and provide a warning that helps
admins correct the problem.
Fixes: 778be232a207 ("NFS do not find client in NFSv4 ...")
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Tested-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/sunrpc/svc.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/net/sunrpc/svc.c b/net/sunrpc/svc.c index 87290a5a9ac7..c5b0cb4f4056 100644 --- a/net/sunrpc/svc.c +++ b/net/sunrpc/svc.c @@ -1194,6 +1194,11 @@ svc_process_common(struct svc_rqst *rqstp, struct kvec *argv, struct kvec *resv) procp->pc_release(rqstp, NULL, rqstp->rq_resp); goto dropit; } + if (*statp == rpc_autherr_badcred) { + if (procp->pc_release) + procp->pc_release(rqstp, NULL, rqstp->rq_resp); + goto err_bad_auth; + } if (*statp == rpc_success && (xdr = procp->pc_encode) && !xdr(rqstp, resv->iov_base+resv->iov_len, rqstp->rq_resp)) { |