scsi: qedi: Fix memory leak in tmf response processing.
authorDupuis, Chad <chad.dupuis@cavium.com>
Fri, 24 Feb 2017 08:31:56 +0000 (00:31 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 28 Feb 2017 03:21:26 +0000 (22:21 -0500)
Signed-off-by: Manish Rangankar <manish.rangankar@cavium.com>
Signed-off-by: Chad Dupuis <chad.dupuis@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qedi/qedi_fw.c

index b1d3904ae8fd8443e6295f19336a294244af4a1e..c9f0ef4e11b33ce9ca2a707645b1b2088b6a1f63 100644 (file)
@@ -165,10 +165,9 @@ static void qedi_tmf_resp_work(struct work_struct *work)
        iscsi_block_session(session->cls_session);
        rval = qedi_cleanup_all_io(qedi, qedi_conn, qedi_cmd->task, true);
        if (rval) {
-               clear_bit(QEDI_CONN_FW_CLEANUP, &qedi_conn->flags);
                qedi_clear_task_idx(qedi, qedi_cmd->task_id);
                iscsi_unblock_session(session->cls_session);
-               return;
+               goto exit_tmf_resp;
        }
 
        iscsi_unblock_session(session->cls_session);
@@ -177,6 +176,8 @@ static void qedi_tmf_resp_work(struct work_struct *work)
        spin_lock(&session->back_lock);
        __iscsi_complete_pdu(conn, (struct iscsi_hdr *)resp_hdr_ptr, NULL, 0);
        spin_unlock(&session->back_lock);
+
+exit_tmf_resp:
        kfree(resp_hdr_ptr);
        clear_bit(QEDI_CONN_FW_CLEANUP, &qedi_conn->flags);
 }