ChangeSet 1.2231.1.173, 2005/03/28 20:04:30-08:00, [EMAIL PROTECTED]
[PATCH] nlm: fix f_count leak
I can't see any reason for this file->f_count++. Removing it fixes a
bug
which leaves an exported filesystem busy if a callback for a lock held
on that
filesystem ever failed.
Found by Terence Rokop.
Signed-off-by: J. Bruce Fields <[EMAIL PROTECTED]>
Signed-off-by: Neil Brown <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
svclock.c | 1 -
1 files changed, 1 deletion(-)
diff -Nru a/fs/lockd/svclock.c b/fs/lockd/svclock.c
--- a/fs/lockd/svclock.c 2005-03-28 21:44:38 -08:00
+++ b/fs/lockd/svclock.c 2005-03-28 21:44:38 -08:00
@@ -641,7 +641,6 @@
} else {
/* Lock is now held by client, or has been rejected.
* In both cases, the block should be removed. */
- file->f_count++;
up(&file->f_sema);
if (status == NLM_LCK_GRANTED)
nlmsvc_delete_block(block, 0);
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
