diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-09-02 07:44:51 -0600 |
|---|---|---|
| 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) | |
| download | lwn-cf93dccea67ad8f5e0d9163c6a0a584550bbd7cd.tar.gz lwn-cf93dccea67ad8f5e0d9163c6a0a584550bbd7cd.zip | |
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, spin_lock(&sdev->stat_lock); sdev->stat.malloc_errors++; spin_unlock(&sdev->stat_lock); + kfree(bbio); return -ENOMEM; } sblock->page_count++; |