IB/isert: fix error return code in isert_alloc_login_buf()
authorWei Yongjun <weiyj.lk@gmail.com>
Sun, 7 Aug 2016 12:20:38 +0000 (12:20 +0000)
committerDoug Ledford <dledford@redhat.com>
Mon, 22 Aug 2016 18:26:55 +0000 (14:26 -0400)
Fix to return error code -ENOMEM from the alloc error handling
case instead of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun <weiyj.lk@gmail.com>
Acked-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/ulp/isert/ib_isert.c

index ba6be060a476b19d608c7797bed4526b7df04332..7914c14478cd190e24d30151a5560f01980e12d2 100644 (file)
@@ -448,7 +448,7 @@ isert_alloc_login_buf(struct isert_conn *isert_conn,
 
        isert_conn->login_rsp_buf = kzalloc(ISER_RX_PAYLOAD_SIZE, GFP_KERNEL);
        if (!isert_conn->login_rsp_buf) {
-               isert_err("Unable to allocate isert_conn->login_rspbuf\n");
+               ret = -ENOMEM;
                goto out_unmap_login_req_buf;
        }