diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-09-02 09:44:51 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-01 15:19:09 -0400 |
commit | cf93dccea67ad8f5e0d9163c6a0a584550bbd7cd (patch) | |
tree | 848f4ec0cc99396d468cbc4addc2707b1064b364 /fs/btrfs/scrub.c | |
parent | 7014cdb49305eda0767d2ae6136f8c191ea8fd81 (diff) |
Btrfs: fix possible memory leak in scrub_setup_recheck_block()
bbio has been malloced in btrfs_map_block() and should be
freed before leaving from the error handling cases.
spatch with a semantic match is used to found this problem.
(http://coccinelle.lip6.fr/)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Diffstat (limited to 'fs/btrfs/scrub.c')
-rw-r--r-- | fs/btrfs/scrub.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index b223620cd5a6..4e9eafe01c55 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c | |||
@@ -1029,6 +1029,7 @@ static int scrub_setup_recheck_block(struct scrub_dev *sdev, | |||
1029 | spin_lock(&sdev->stat_lock); | 1029 | spin_lock(&sdev->stat_lock); |
1030 | sdev->stat.malloc_errors++; | 1030 | sdev->stat.malloc_errors++; |
1031 | spin_unlock(&sdev->stat_lock); | 1031 | spin_unlock(&sdev->stat_lock); |
1032 | kfree(bbio); | ||
1032 | return -ENOMEM; | 1033 | return -ENOMEM; |
1033 | } | 1034 | } |
1034 | sblock->page_count++; | 1035 | sblock->page_count++; |