Running on Coverity produced the following errors:
- coding style (indentation)
- memset size mismatch errors
  note: comment cases where it is purposely a mismatch

Fix the errors

Signed-off-by: Dick Kennedy <dick.kenn...@broadcom.com>
Signed-off-by: James Smart <jsmart2...@gmail.com>
---
 drivers/scsi/lpfc/lpfc_els.c   | 10 +++++-----
 drivers/scsi/lpfc/lpfc_nvme.c  |  5 +++--
 drivers/scsi/lpfc/lpfc_nvmet.c |  2 +-
 drivers/scsi/lpfc/lpfc_sli.c   |  2 +-
 4 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/drivers/scsi/lpfc/lpfc_els.c b/drivers/scsi/lpfc/lpfc_els.c
index d919f3161160..54ec7f0822e5 100644
--- a/drivers/scsi/lpfc/lpfc_els.c
+++ b/drivers/scsi/lpfc/lpfc_els.c
@@ -5667,16 +5667,16 @@ lpfc_rdp_res_attach_port_names(struct 
fc_rdp_port_name_desc *desc,
        desc->tag = cpu_to_be32(RDP_PORT_NAMES_DESC_TAG);
        if (vport->fc_flag & FC_FABRIC) {
                memcpy(desc->port_names.wwnn, &vport->fabric_nodename,
-                               sizeof(desc->port_names.wwnn));
+                      sizeof(desc->port_names.wwnn));
 
                memcpy(desc->port_names.wwpn, &vport->fabric_portname,
-                               sizeof(desc->port_names.wwpn));
+                      sizeof(desc->port_names.wwpn));
        } else {  /* Point to Point */
                memcpy(desc->port_names.wwnn, &ndlp->nlp_nodename,
-                               sizeof(desc->port_names.wwnn));
+                      sizeof(desc->port_names.wwnn));
 
-               memcpy(desc->port_names.wwnn, &ndlp->nlp_portname,
-                               sizeof(desc->port_names.wwpn));
+               memcpy(desc->port_names.wwpn, &ndlp->nlp_portname,
+                      sizeof(desc->port_names.wwpn));
        }
 
        desc->length = cpu_to_be32(sizeof(desc->port_names));
diff --git a/drivers/scsi/lpfc/lpfc_nvme.c b/drivers/scsi/lpfc/lpfc_nvme.c
index c7f5b50c3820..41b124b69947 100644
--- a/drivers/scsi/lpfc/lpfc_nvme.c
+++ b/drivers/scsi/lpfc/lpfc_nvme.c
@@ -436,6 +436,7 @@ lpfc_nvme_gen_req(struct lpfc_vport *vport, struct 
lpfc_dmabuf *bmp,
                return 1;
 
        wqe = &genwqe->wqe;
+       /* Initialize only 64 bytes */
        memset(wqe, 0, sizeof(union lpfc_wqe));
 
        genwqe->context3 = (uint8_t *)bmp;
@@ -1855,7 +1856,7 @@ lpfc_nvme_fcp_abort(struct nvme_fc_local_port 
*pnvme_lport,
        /* WQEs are reused.  Clear stale data and set key fields to
         * zero like ia, iaab, iaar, xri_tag, and ctxt_tag.
         */
-       memset(abts_wqe, 0, sizeof(union lpfc_wqe));
+       memset(abts_wqe, 0, sizeof(*abts_wqe));
        bf_set(abort_cmd_criteria, &abts_wqe->abort_cmd, T_XRI_TAG);
 
        /* word 7 */
@@ -1982,7 +1983,7 @@ lpfc_get_nvme_buf(struct lpfc_hba *phba, struct 
lpfc_nodelist *ndlp,
                sgl->word2 = cpu_to_le32(sgl->word2);
                /* Fill in word 3 / sgl_len during cmd submission */
 
-               /* Initialize WQE */
+               /* Initialize 64 bytes only */
                memset(wqe, 0, sizeof(union lpfc_wqe));
 
                if (lpfc_ndlp_check_qdepth(phba, ndlp)) {
diff --git a/drivers/scsi/lpfc/lpfc_nvmet.c b/drivers/scsi/lpfc/lpfc_nvmet.c
index f0840e3182c0..f42cc3150c6f 100644
--- a/drivers/scsi/lpfc/lpfc_nvmet.c
+++ b/drivers/scsi/lpfc/lpfc_nvmet.c
@@ -3330,7 +3330,7 @@ lpfc_nvmet_sol_fcp_issue_abort(struct lpfc_hba *phba,
        /* WQEs are reused.  Clear stale data and set key fields to
         * zero like ia, iaab, iaar, xri_tag, and ctxt_tag.
         */
-       memset(abts_wqe, 0, sizeof(union lpfc_wqe));
+       memset(abts_wqe, 0, sizeof(*abts_wqe));
 
        /* word 3 */
        bf_set(abort_cmd_criteria, &abts_wqe->abort_cmd, T_XRI_TAG);
diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c
index 6ac6620aab00..0142545873a5 100644
--- a/drivers/scsi/lpfc/lpfc_sli.c
+++ b/drivers/scsi/lpfc/lpfc_sli.c
@@ -9815,7 +9815,7 @@ lpfc_sli4_iocb2wqe(struct lpfc_hba *phba, struct 
lpfc_iocbq *iocbq,
                 * we re-construct this WQE here based on information in
                 * iocbq from scratch.
                 */
-               memset(wqe, 0, sizeof(union lpfc_wqe));
+               memset(wqe, 0, sizeof(*wqe));
                /* OX_ID is invariable to who sent ABTS to CT exchange */
                bf_set(xmit_bls_rsp64_oxid, &wqe->xmit_bls_rsp,
                       bf_get(lpfc_abts_oxid, &iocbq->iocb.un.bls_rsp));
-- 
2.13.7

Reply via email to