[PATCH 2/2] zram: fix memory freeing in zram_bvec_read

From: Geliang Tang
Date: Sun Nov 08 2015 - 10:35:04 EST


When user_mem is NULL, there is no need to kunmap_atomic it.

Signed-off-by: Geliang Tang <geliangtang@xxxxxxx>
---
drivers/block/zram/zram_drv.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
index 197a7ad..da7b348 100644
--- a/drivers/block/zram/zram_drv.c
+++ b/drivers/block/zram/zram_drv.c
@@ -625,7 +625,7 @@ static int zram_bvec_read(struct zram *zram, struct bio_vec *bvec,
if (!uncmem) {
pr_err("Unable to allocate temp memory\n");
ret = -ENOMEM;
- goto out_cleanup;
+ goto out_free;
}

ret = zram_decompress_page(zram, uncmem, index);
@@ -641,6 +641,7 @@ static int zram_bvec_read(struct zram *zram, struct bio_vec *bvec,
ret = 0;
out_cleanup:
kunmap_atomic(user_mem);
+out_free:
if (is_partial_io(bvec))
kfree(uncmem);
return ret;
--
2.5.0


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/