> 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.richard...@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;
LGTM.
Acked-by: Ian Stokes <ian.sto...@intel.com>
> --
> 2.43.0
