diff options
author | Adrian Hunter <[ext-adrian.hunter@nokia.com]> | 2007-01-22 03:01:01 -0500 |
---|---|---|
committer | Kyungmin Park <kyungmin.park@samsung.com> | 2007-01-22 07:27:56 -0500 |
commit | f00b0046d2eafac3e78e8def9374c7492820a9d2 (patch) | |
tree | ddcbbacccff7f64fdac41cdfe91a07b6aca20aca /drivers/mtd/onenand/onenand_base.c | |
parent | 75384b0d9c04dc2d48f45825f84a982eaf5c2f53 (diff) |
[MTD] OneNAND: Free the bad block table when the device is released
OneNAND does 2 memory allocations for bad block information.
Only one of them was being freed.
Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Diffstat (limited to 'drivers/mtd/onenand/onenand_base.c')
-rw-r--r-- | drivers/mtd/onenand/onenand_base.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/mtd/onenand/onenand_base.c b/drivers/mtd/onenand/onenand_base.c index 0ade23749ee1..0249b4aa0976 100644 --- a/drivers/mtd/onenand/onenand_base.c +++ b/drivers/mtd/onenand/onenand_base.c | |||
@@ -2133,8 +2133,11 @@ void onenand_release(struct mtd_info *mtd) | |||
2133 | del_mtd_device (mtd); | 2133 | del_mtd_device (mtd); |
2134 | 2134 | ||
2135 | /* Free bad block table memory, if allocated */ | 2135 | /* Free bad block table memory, if allocated */ |
2136 | if (this->bbm) | 2136 | if (this->bbm) { |
2137 | struct bbm_info *bbm = this->bbm; | ||
2138 | kfree(bbm->bbt); | ||
2137 | kfree(this->bbm); | 2139 | kfree(this->bbm); |
2140 | } | ||
2138 | /* Buffer allocated by onenand_scan */ | 2141 | /* Buffer allocated by onenand_scan */ |
2139 | if (this->options & ONENAND_PAGEBUF_ALLOC) | 2142 | if (this->options & ONENAND_PAGEBUF_ALLOC) |
2140 | kfree(this->page_buf); | 2143 | kfree(this->page_buf); |