Commit 57df81e3 authored by Sagi Grimberg's avatar Sagi Grimberg Committed by Nicholas Bellinger

iser-target: Remove redundant check on the device

In iser_connect_release there is no chance that
the iser device is set to NULL, if this happens
we have a BUG. So use BUG_ON.
Signed-off-by: default avatarSagi Grimberg <sagig@mellanox.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent c6b8e918
...@@ -797,7 +797,9 @@ isert_connect_release(struct isert_conn *isert_conn) ...@@ -797,7 +797,9 @@ isert_connect_release(struct isert_conn *isert_conn)
isert_dbg("conn %p\n", isert_conn); isert_dbg("conn %p\n", isert_conn);
if (device && device->use_fastreg) BUG_ON(!device);
if (device->use_fastreg)
isert_conn_free_fastreg_pool(isert_conn); isert_conn_free_fastreg_pool(isert_conn);
isert_free_rx_descriptors(isert_conn); isert_free_rx_descriptors(isert_conn);
...@@ -814,10 +816,9 @@ isert_connect_release(struct isert_conn *isert_conn) ...@@ -814,10 +816,9 @@ isert_connect_release(struct isert_conn *isert_conn)
if (isert_conn->login_buf) if (isert_conn->login_buf)
isert_free_login_buf(isert_conn); isert_free_login_buf(isert_conn);
kfree(isert_conn); isert_device_put(device);
if (device) kfree(isert_conn);
isert_device_put(device);
} }
static void static void
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment