From: Shiqi Liu <835703...@qq.com>

As the possible failure of the malloc(), the not_checked and
checked could be NULL pointer.
Therefore, it should be better to check it in order to avoid
the dereference of the NULL pointer.

Fixes: fa8054c8c88 ("lib/node: add ethdev control")
Signed-off-by: Shiqi Liu <835703...@qq.com>
---
 lib/node/ethdev_ctrl.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lib/node/ethdev_ctrl.c b/lib/node/ethdev_ctrl.c
index 5294607619..eef0785c21 100644
--- a/lib/node/ethdev_ctrl.c
+++ b/lib/node/ethdev_ctrl.c
@@ -75,6 +75,9 @@ rte_node_eth_config(struct rte_node_ethdev_config *conf, 
uint16_t nb_confs,
 
                        /* Add it to list of ethdev rx nodes for lookup */
                        elem = malloc(sizeof(ethdev_rx_node_elem_t));
+                       if (elem == NULL) {
+                               return -ENOMEM;
+                       }
                        memset(elem, 0, sizeof(ethdev_rx_node_elem_t));
                        elem->ctx.port_id = port_id;
                        elem->ctx.queue_id = j;
-- 
2.35.1.windows.2

Reply via email to