[PATCH] bfa: Fix to set correct return error codes and misc cleanup.

[kgudipat]

From: Krishna Gudipati <kgudi...@brocade.com>

Change details:
        - Remove unnecessary if NULL check in function bfa_fcs_vport_free().
        - Set correct return error codes in case of memory allocation failure
          in the BSG ELS/CT passthru command handler.

Signed-off-by: Krishna Gudipati <kgudi...@brocade.com>
---
 drivers/scsi/bfa/bfa_fcs_lport.c |    4 +---
 drivers/scsi/bfa/bfad_bsg.c      |    5 ++++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/bfa/bfa_fcs_lport.c b/drivers/scsi/bfa/bfa_fcs_lport.c
index 937000d..bcc4966 100644
--- a/drivers/scsi/bfa/bfa_fcs_lport.c
+++ b/drivers/scsi/bfa/bfa_fcs_lport.c
@@ -5722,9 +5722,7 @@ bfa_fcs_vport_free(struct bfa_fcs_vport_s *vport)
         * The memory for the bfad_vport_s is freed from the FC function
         * template vport_delete entry point.
         */
-       if (vport_drv)
-               bfad_im_port_delete(vport_drv->drv_port.bfad,
-                               &vport_drv->drv_port);
+       bfad_im_port_delete(vport_drv->drv_port.bfad, &vport_drv->drv_port);
 }
 
 /*
diff --git a/drivers/scsi/bfa/bfad_bsg.c b/drivers/scsi/bfa/bfad_bsg.c
index e1f4b10..9c1495b 100644
--- a/drivers/scsi/bfa/bfad_bsg.c
+++ b/drivers/scsi/bfa/bfad_bsg.c
@@ -3008,12 +3008,15 @@ bfad_im_bsg_els_ct_request(struct fc_bsg_job *job)
         * buffer of size bsg_data->payload_len
         */
        bsg_fcpt = kzalloc(bsg_data->payload_len, GFP_KERNEL);
-       if (!bsg_fcpt)
+       if (!bsg_fcpt) {
+               rc = -ENOMEM;
                goto out;
+       }
 
        if (copy_from_user((uint8_t *)bsg_fcpt, bsg_data->payload,
                                bsg_data->payload_len)) {
                kfree(bsg_fcpt);
+               rc = -EIO;
                goto out;
        }
 
-- 
1.7.3.rc1

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