aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@fieldses.org>2005-11-10 19:08:00 -0500
committerTrond Myklebust <Trond.Myklebust@netapp.com>2005-11-13 12:29:54 -0500
commitdc15ae14e97ee9d5ed740cbb0b94996076d8b37e (patch)
treea159edc76dedf81d73f17d844fbf965d5ded9f39
parent9e6c67fd2716720d9029d38ea25884efcfdedeb6 (diff)
downloadlinux-dc15ae14e97ee9d5ed740cbb0b94996076d8b37e.tar.gz
[PATCH] VFS: Fix memory leak with file leases
The patch http://linux.bkbits.net:8080/linux-2.6/diffs/fs/locks.c@1.70??nav=index.html introduced a pretty nasty memory leak in the lease code. When freeing the lease, the code in locks_delete_lock() will correctly clean up the fasync queue, but when we return to fcntl_setlease(), the freed fasync entry will be reinstated. This patch ensures that we skip the call to fasync_helper() when we're freeing up the lease. Signed-off-by: J. Bruce Fields <bfields@fieldses.org> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r--fs/locks.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/locks.c b/fs/locks.c
index a1e8b2248014c0..600d1fbe35710e 100644
--- a/fs/locks.c
+++ b/fs/locks.c
@@ -1430,7 +1430,7 @@ int fcntl_setlease(unsigned int fd, struct file *filp, long arg)
lock_kernel();
error = __setlease(filp, arg, &flp);
- if (error)
+ if (error || arg == F_UNLCK)
goto out_unlock;
error = fasync_helper(fd, filp, 1, &flp->fl_fasync);