diff options
| author | Oleg Drokin <Oleg.Drokin@Sun.COM> | 2008-01-11 21:57:35 -0500 |
|---|---|---|
| committer | J. Bruce Fields <bfields@citi.umich.edu> | 2008-02-01 16:42:07 -0500 |
| commit | 54ca95eb362d6988a577965ffb77c08702adb890 (patch) | |
| tree | a18ff136ce9d530eaa90a42831d9b85a91ab6321 /fs/lockd/svclock.c | |
| parent | 8838dc43d6544570e8969a74ddc4a0d21abffde6 (diff) | |
| download | lwn-54ca95eb362d6988a577965ffb77c08702adb890.tar.gz lwn-54ca95eb362d6988a577965ffb77c08702adb890.zip | |
Leak in nlmsvc_testlock for async GETFL case
Fix nlm_block leak for the case of supplied blocking lock info.
Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/lockd/svclock.c')
| -rw-r--r-- | fs/lockd/svclock.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/lockd/svclock.c b/fs/lockd/svclock.c index 84c4d5e04ebb..2f4d8fa66689 100644 --- a/fs/lockd/svclock.c +++ b/fs/lockd/svclock.c @@ -505,12 +505,12 @@ nlmsvc_testlock(struct svc_rqst *rqstp, struct nlm_file *file, goto out; } if (block->b_flags & B_GOT_CALLBACK) { + nlmsvc_unlink_block(block); if (block->b_fl != NULL && block->b_fl->fl_type != F_UNLCK) { lock->fl = *block->b_fl; goto conf_lock; } else { - nlmsvc_unlink_block(block); ret = nlm_granted; goto out; } |