[PATCH 07/11] net/enetfec: fix memory leak in Rx buffer cleanup

[Hemant Agrawal]

The RX buffer cleanup logic did not check for NULL before freeing mbufs,
which could lead to undefined behavior. This patch adds a NULL check
before freeing each mbuf.

Fixes: ecae71571b0d ("net/enetfec: support Rx/Tx")
Cc: sta...@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>
---
 drivers/net/enetfec/enet_ethdev.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/net/enetfec/enet_ethdev.c 
b/drivers/net/enetfec/enet_ethdev.c
index bcecab828e..60bb4f7ebd 100644
--- a/drivers/net/enetfec/enet_ethdev.c
+++ b/drivers/net/enetfec/enet_ethdev.c
@@ -1,5 +1,5 @@
 /* SPDX-License-Identifier: BSD-3-Clause
- * Copyright 2020-2021,2023 NXP
+ * Copyright 2020-2021,2023-2024 NXP
  */
 
 #include <inttypes.h>
@@ -172,8 +172,10 @@ enet_free_buffers(struct rte_eth_dev *dev)
                bdp = rxq->bd.base;
                for (i = 0; i < rxq->bd.ring_size; i++) {
                        mbuf = rxq->rx_mbuf[i];
-                       rxq->rx_mbuf[i] = NULL;
-                       rte_pktmbuf_free(mbuf);
+                       if (mbuf) {
+                               rxq->rx_mbuf[i] = NULL;
+                               rte_pktmbuf_free(mbuf);
+                       }
                        bdp = enet_get_nextdesc(bdp, &rxq->bd);
                }
        }
@@ -547,7 +549,7 @@ enetfec_rx_queue_setup(struct rte_eth_dev *dev,
                }
        }
        rte_free(rxq);
-       return errno;
+       return -ENOMEM;
 }
 
 static const struct eth_dev_ops enetfec_ops = {
-- 
2.25.1