diff options
| author | Oleg Drokin <green@linuxhacker.ru> | 2016-07-15 06:20:22 +0300 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2016-08-05 00:11:46 +0300 |
| commit | 7eed34f18d9f0f96f51bfb73d649a51a20857eb0 (patch) | |
| tree | 32d4f2332f9305226f81d60f262842f5b9250b35 /fs/nfsd/nfs4proc.c | |
| parent | c7995f8a70c441146e128934d775d4ba0e4d7bc4 (diff) | |
| download | linux-7eed34f18d9f0f96f51bfb73d649a51a20857eb0.tar.xz | |
nfsd: Make creates return EEXIST instead of EACCES
When doing a create (mkdir/mknod) on a name, it's worth
checking the name exists first before returning EACCES in case
the directory is not writeable by the user.
This makes return values on the client more consistent
regardless of whenever the entry there is cached in the local
cache or not.
Another positive side effect is certain programs only expect
EEXIST in that case even despite POSIX allowing any valid
error to be returned.
Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4proc.c')
| -rw-r--r-- | fs/nfsd/nfs4proc.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c index e0c15f879d89..9d7e1edf0cca 100644 --- a/fs/nfsd/nfs4proc.c +++ b/fs/nfsd/nfs4proc.c @@ -605,8 +605,12 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, fh_init(&resfh, NFS4_FHSIZE); + /* + * We just check that parent is accessible here, nfsd_* do their + * own access permission checks + */ status = fh_verify(rqstp, &cstate->current_fh, S_IFDIR, - NFSD_MAY_CREATE); + NFSD_MAY_EXEC); if (status) return status; |