diff options
| author | Fred Isaman <iisaman@netapp.com> | 2011-06-15 12:31:02 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-06-15 12:39:23 -0400 |
| commit | ea0ded748bdea78f9e2fefb571f7d6ce9edb4f89 (patch) | |
| tree | ce4589aecc27e8936529ce1cbc4d8a129a1f43b2 /fs/nfs | |
| parent | 1ed3a8539af7b36aa5c977f304e80f7fc8d27bfc (diff) | |
| download | op-kernel-dev-ea0ded748bdea78f9e2fefb571f7d6ce9edb4f89.zip op-kernel-dev-ea0ded748bdea78f9e2fefb571f7d6ce9edb4f89.tar.gz | |
nfs4.1: prevent race that allowed use of freed layout in _pnfs_return_layout
mark_matching_lsegs_invalid could put the last ref to the layout, so
the get_layout_hdr needs to be called first.
Signed-off-by: Fred Isaman <iisaman@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs')
| -rw-r--r-- | fs/nfs/pnfs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index 8f95822..730d4db 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c @@ -640,10 +640,10 @@ _pnfs_return_layout(struct inode *ino) return status; } stateid = nfsi->layout->plh_stateid; - mark_matching_lsegs_invalid(lo, &tmp_list, NULL); - lo->plh_block_lgets++; /* Reference matched in nfs4_layoutreturn_release */ get_layout_hdr(lo); + mark_matching_lsegs_invalid(lo, &tmp_list, NULL); + lo->plh_block_lgets++; spin_unlock(&ino->i_lock); pnfs_free_lseg_list(&tmp_list); |
