scsi: qedf: Add missing skb frees in error path

Signed-off-by: Chad Dupuis <chad.dupuis@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/qedf/qedf_fip.c b/drivers/scsi/qedf/qedf_fip.c
index 773558fc0697513a523eeae10b6598eb37ea22a9..aef199c12fb4cf249a0e3b51455cd1cdd8a64b45 100644 (file)
--- a/drivers/scsi/qedf/qedf_fip.c
+++ b/drivers/scsi/qedf/qedf_fip.c
@@ -181,6 +181,7 @@ void qedf_fip_recv(struct qedf_ctx *qedf, struct sk_buff *skb)
                        QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC,
                            "Dropping CVL since FCF has not been selected "
                            "yet.");
+                       kfree_skb(skb);
                        return;
                }
 

diff --git a/drivers/scsi/qedf/qedf_main.c b/drivers/scsi/qedf/qedf_main.c
index e14b402f25d592d0a3c89255acf4192794f56992..8dcacc5b5f604c50ca46bf3190ddfb12bb019cb7 100644 (file)
--- a/drivers/scsi/qedf/qedf_main.c
+++ b/drivers/scsi/qedf/qedf_main.c
@@ -2190,6 +2190,7 @@ static void qedf_recv_frame(struct qedf_ctx *qedf,
        if (ntoh24(&dest_mac[3]) != ntoh24(fh->fh_d_id)) {
                QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2,
                    "FC frame d_id mismatch with MAC %pM.\n", dest_mac);
+               kfree_skb(skb);
                return;
        }