diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2013-05-03 16:09:09 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2013-05-30 14:34:51 +0100 |
| commit | 26bdeae55b558d983d7c6033200eac14f52f063d (patch) | |
| tree | 8fc7f6fa3acea1448b9fe318d60b34cb5809093e /fs/nfsd/nfs4idmap.c | |
| parent | 8e590dc654420b9aa0172b6a3ff64a7675927672 (diff) | |
| download | kernel_samsung_smdk4412-26bdeae55b558d983d7c6033200eac14f52f063d.zip kernel_samsung_smdk4412-26bdeae55b558d983d7c6033200eac14f52f063d.tar.gz kernel_samsung_smdk4412-26bdeae55b558d983d7c6033200eac14f52f063d.tar.bz2 | |
nfsd4: don't allow owner override on 4.1 CLAIM_FH opens
commit 9f415eb25574db4b73a9a712a4438e41dc284922 upstream.
The Linux client is using CLAIM_FH to implement regular opens, not just
recovery cases, so it depends on the server to check permissions
correctly.
Therefore the owner override, which may make sense in the delegation
recovery case, isn't right in the CLAIM_FH case.
Symptoms: on a client with 49f9a0fafd844c32f2abada047c0b9a5ba0d6255
"NFSv4.1: Enable open-by-filehandle", Bryan noticed this:
touch test.txt
chmod 000 test.txt
echo test > test.txt
succeeding.
Reported-by: Bryan Schumaker <bjschuma@netapp.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'fs/nfsd/nfs4idmap.c')
0 files changed, 0 insertions, 0 deletions