kfree on null pointer is a no-op.

Signed-off-by: Sachin Kamat <[email protected]>
---
 drivers/target/iscsi/iscsi_target_login.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/target/iscsi/iscsi_target_login.c 
b/drivers/target/iscsi/iscsi_target_login.c
index 7bf9ccc..fdb632f 100644
--- a/drivers/target/iscsi/iscsi_target_login.c
+++ b/drivers/target/iscsi/iscsi_target_login.c
@@ -1118,10 +1118,8 @@ new_sess_out:
                idr_remove(&sess_idr, conn->sess->session_index);
                spin_unlock_bh(&sess_idr_lock);
        }
-       if (conn->sess->sess_ops)
-               kfree(conn->sess->sess_ops);
-       if (conn->sess)
-               kfree(conn->sess);
+       kfree(conn->sess->sess_ops);
+       kfree(conn->sess);
 old_sess_out:
        iscsi_stop_login_thread_timer(np);
        /*
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to