DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] net/ice: add check for null-pointer dereference
@ 2024-11-14 11:35 Bruce Richardson
  0 siblings, 0 replies; only message in thread
From: Bruce Richardson @ 2024-11-14 11:35 UTC (permalink / raw)
  To: dev; +Cc: Bruce Richardson

Although it should never occur that the teid (scheduler node id) value
for a txq is incorrect, put in a check for null following the scheduler
node lookup in ice_tm_setup_txq_node. This provides some additional
safety and should eliminate a coverity issue too.

Coverity issue: 448957
Fixes: 715d449a965b ("net/ice: enhance Tx scheduler hierarchy support")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/net/ice/ice_tm.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/net/ice/ice_tm.c b/drivers/net/ice/ice_tm.c
index 18ac324a61..ff3a6cd77f 100644
--- a/drivers/net/ice/ice_tm.c
+++ b/drivers/net/ice/ice_tm.c
@@ -679,6 +679,10 @@ ice_tm_setup_txq_node(struct ice_pf *pf, struct ice_hw *hw, uint16_t qid, uint32
 	struct ice_sched_node *hw_node = ice_sched_find_node_by_teid(hw->port_info->root, teid);
 	struct ice_tm_node *sw_node = find_node(pf->tm_conf.root, qid);
 
+	/* bad node teid passed */
+	if (hw_node == NULL)
+		return -ENOENT;
+
 	/* not configured in hierarchy */
 	if (sw_node == NULL)
 		return 0;
-- 
2.43.0


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-11-14 11:35 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-14 11:35 [PATCH] net/ice: add check for null-pointer dereference Bruce Richardson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).