diff options
| author | Trond Myklebust <trond.myklebust@primarydata.com> | 2017-07-17 10:11:11 -0400 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2017-08-15 11:54:46 -0400 |
| commit | 7cb9cd9aa2eafe869935d4168031f1ed376d924c (patch) | |
| tree | 4c4c2028f5f65e0f5df1c0929a087e21eed7db21 /fs | |
| parent | 08fead2ae5a9953d47677416cc5f6bcae448480d (diff) | |
| download | op-kernel-dev-7cb9cd9aa2eafe869935d4168031f1ed376d924c.zip op-kernel-dev-7cb9cd9aa2eafe869935d4168031f1ed376d924c.tar.gz | |
NFS: Fix a reference and lock leak in nfs_lock_and_join_requests()
Yes, this is a situation that should never happen (hence the WARN_ON)
but we should still ensure that we free up the locks and references to
the faulty pages.
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/nfs/write.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index bb01909..1ca7597 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c @@ -526,8 +526,7 @@ try_again: } else if (WARN_ON_ONCE(subreq->wb_offset < head->wb_offset || ((subreq->wb_offset + subreq->wb_bytes) > (head->wb_offset + total_bytes)))) { - nfs_page_group_unlock(head); - spin_unlock(&inode->i_lock); + nfs_unroll_locks_and_wait(inode, head, subreq); return ERR_PTR(-EIO); } |
