diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2015-09-01 19:40:53 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2015-09-01 19:53:40 +0200 |
| commit | f984a7ce58ea9a12eca7f960bdf68124c8589b60 (patch) | |
| tree | 23f161f7d054798bcb1b1d614cefdbae71997979 | |
| parent | nfsd: ensure that delegation stateid hash references are only put once (diff) | |
| download | linux-f984a7ce58ea9a12eca7f960bdf68124c8589b60.tar.xz linux-f984a7ce58ea9a12eca7f960bdf68124c8589b60.zip | |
nfsd: return CLID_INUSE for unexpected SETCLIENTID_CONFIRM case
Somebody with a Solaris client was hitting this case. We haven't
figured out why yet, and don't have a reproducer. Meanwhile Frank
noticed that RFC 7530 actually recommends CLID_INUSE for this case.
Unlikely to help the original reporter, but may as well fix it.
Reported-by: Frank Filz <ffilzlnx@mindspring.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
| -rw-r--r-- | fs/nfsd/nfs4state.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 416f32e34a33..08746ec1d44a 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c @@ -3111,10 +3111,11 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, /* * We try hard to give out unique clientid's, so if we get an * attempt to confirm the same clientid with a different cred, - * there's a bug somewhere. Let's charitably assume it's our - * bug. + * the client may be buggy; this should never happen. + * + * Nevertheless, RFC 7530 recommends INUSE for this case: */ - status = nfserr_serverfault; + status = nfserr_clid_inuse; if (unconf && !same_creds(&unconf->cl_cred, &rqstp->rq_cred)) goto out; if (conf && !same_creds(&conf->cl_cred, &rqstp->rq_cred)) |