diff options
| author | Trond Myklebust <trond.myklebust@hammerspace.com> | 2019-10-22 14:35:57 +0200 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@hammerspace.com> | 2019-11-04 03:28:45 +0100 |
| commit | 5decae1623f5a1e409ed6befdc7c03ce53e8cd09 (patch) | |
| tree | 77aa7cd6f578691726ceabeb33f8a3ad4ecab664 /fs/nfs | |
| parent | NFSv4: Fix delegation handling in update_open_stateid() (diff) | |
| download | linux-5decae1623f5a1e409ed6befdc7c03ce53e8cd09.tar.xz linux-5decae1623f5a1e409ed6befdc7c03ce53e8cd09.zip | |
NFSv4: nfs4_callback_getattr() should ignore revoked delegations
If the delegation has been revoked, ignore it.
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Diffstat (limited to 'fs/nfs')
| -rw-r--r-- | fs/nfs/callback_proc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/nfs/callback_proc.c b/fs/nfs/callback_proc.c index db3e7771e597..cd4c6bc81cae 100644 --- a/fs/nfs/callback_proc.c +++ b/fs/nfs/callback_proc.c @@ -26,7 +26,6 @@ __be32 nfs4_callback_getattr(void *argp, void *resp, struct cb_getattrargs *args = argp; struct cb_getattrres *res = resp; struct nfs_delegation *delegation; - struct nfs_inode *nfsi; struct inode *inode; res->status = htonl(NFS4ERR_OP_NOT_IN_SESSION); @@ -47,9 +46,8 @@ __be32 nfs4_callback_getattr(void *argp, void *resp, -ntohl(res->status)); goto out; } - nfsi = NFS_I(inode); rcu_read_lock(); - delegation = rcu_dereference(nfsi->delegation); + delegation = nfs4_get_valid_delegation(inode); if (delegation == NULL || (delegation->type & FMODE_WRITE) == 0) goto out_iput; res->size = i_size_read(inode); |