Messages in this thread Patch in this message | | | From | Geliang Tang <> | Subject | [PATCH 2/2] zram: fix memory freeing in zram_bvec_read | Date | Sun, 8 Nov 2015 23:32:19 +0800 |
| |
When user_mem is NULL, there is no need to kunmap_atomic it.
Signed-off-by: Geliang Tang <geliangtang@163.com> --- 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
| |