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

Signed-off-by: Geliang Tang <[email protected]>
---
 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to