From: Dan Carpenter <dan.carpenter@oracle.com>
Date: Fri, 12 Oct 2012 04:22:43 +0000 (+1100)
Subject: memstick: memory leak on error in msb_ftl_scan()
X-Git-Tag: next-20121016~2^2~67
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=d95cd749e6e9485b2d32eb0586c83b0ff8679f31;p=karo-tx-linux.git

memstick: memory leak on error in msb_ftl_scan()

We need to free "overwrite_flags" before returning.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Maxim Levitsky <maximlevitsly@gmail.com>
Cc: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

diff --git a/drivers/memstick/core/ms_block.c b/drivers/memstick/core/ms_block.c
index 4a751c9e5eff..af191797b363 100644
--- a/drivers/memstick/core/ms_block.c
+++ b/drivers/memstick/core/ms_block.c
@@ -1408,8 +1408,10 @@ static int msb_ftl_scan(struct msb_data *msb)
 			msb_mark_block_used(msb, pba);
 			msb_erase_block(msb, pba);
 			continue;
-		} else if (error)
+		} else if (error) {
+			kfree(overwrite_flags);
 			return error;
+		}
 
 		lba = be16_to_cpu(extra.logical_address);
 		managment_flag = extra.management_flag;