From: Steve Hodgson <steve@xxxxxxxxxxxxxxx> Signed-off-by: Steve Hodgson <steve@xxxxxxxxxxxxxxx> --- drivers/scsi/qla2xxx/qla_bsg.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_bsg.c b/drivers/scsi/qla2xxx/qla_bsg.c index 765be8e..f465c3e 100644 --- a/drivers/scsi/qla2xxx/qla_bsg.c +++ b/drivers/scsi/qla2xxx/qla_bsg.c @@ -726,7 +726,7 @@ qla2x00_process_loopback(struct fc_bsg_job *bsg_job) "Get port config failed.\n"); bsg_job->reply->result = (DID_ERROR << 16); rval = -EPERM; - goto done_free_dma_req; + goto done_free_dma_rsp; } if (elreq.options != EXTERNAL_LOOPBACK) { @@ -740,7 +740,7 @@ qla2x00_process_loopback(struct fc_bsg_job *bsg_job) bsg_job->reply->result = (DID_ERROR << 16); rval = -EPERM; - goto done_free_dma_req; + goto done_free_dma_rsp; } } else { /* For external loopback to work @@ -751,7 +751,7 @@ qla2x00_process_loopback(struct fc_bsg_job *bsg_job) bsg_job->reply->result = (DID_ERROR << 16); rval = -EPERM; - goto done_free_dma_req; + goto done_free_dma_rsp; } } @@ -786,7 +786,7 @@ qla2x00_process_loopback(struct fc_bsg_job *bsg_job) bsg_job->reply->result = (DID_ERROR << 16); rval = -EIO; - goto done_free_dma_req; + goto done_free_dma_rsp; } } else { type = "FC_BSG_HST_VENDOR_LOOPBACK"; @@ -828,6 +828,7 @@ qla2x00_process_loopback(struct fc_bsg_job *bsg_job) rsp_data_len); } +done_free_dma_rsp: dma_free_coherent(&ha->pdev->dev, rsp_data_len, rsp_data, rsp_data_dma); done_free_dma_req: -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html