diff options
| author | Jeff Layton <jlayton@kernel.org> | 2022-07-26 08:45:30 +0200 |
|---|---|---|
| committer | Chuck Lever <chuck.lever@oracle.com> | 2022-07-30 02:17:00 +0200 |
| commit | 876c553cb41026cb6ad3cef970a35e5f69c42a25 (patch) | |
| tree | f9b8ac70320658bc03187aa1c08d117f4e2f8145 /fs/nfsd/vfs.h | |
| parent | NFSD: drop fh argument from alloc_init_deleg (diff) | |
| download | linux-876c553cb41026cb6ad3cef970a35e5f69c42a25.tar.xz linux-876c553cb41026cb6ad3cef970a35e5f69c42a25.zip | |
NFSD: verify the opened dentry after setting a delegation
Between opening a file and setting a delegation on it, someone could
rename or unlink the dentry. If this happens, we do not want to grant a
delegation on the open.
On a CLAIM_NULL open, we're opening by filename, and we may (in the
non-create case) or may not (in the create case) be holding i_rwsem
when attempting to set a delegation. The latter case allows a
race.
After getting a lease, redo the lookup of the file being opened and
validate that the resulting dentry matches the one in the open file
description.
To properly redo the lookup we need an rqst pointer to pass to
nfsd_lookup_dentry(), so make sure that is available.
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'fs/nfsd/vfs.h')
0 files changed, 0 insertions, 0 deletions