[PATCH 1/4] net/ice: remove unnecessary null check

Bruce Richardson bruce.richardson at intel.com
Wed Apr 1 18:44:00 CEST 2026


On Wed, Apr 01, 2026 at 09:40:22AM -0700, Stephen Hemminger wrote:
> No need to check for null before calling free.
> Found by coccinelle script null_free.
> 
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> ---
>  drivers/net/intel/ice/ice_dcf_sched.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
Acked-by: Bruce Richardson <bruce.richardson at intel.com>


> diff --git a/drivers/net/intel/ice/ice_dcf_sched.c b/drivers/net/intel/ice/ice_dcf_sched.c
> index 948774a282..14e902edbe 100644
> --- a/drivers/net/intel/ice/ice_dcf_sched.c
> +++ b/drivers/net/intel/ice/ice_dcf_sched.c
> @@ -877,10 +877,8 @@ static int ice_dcf_hierarchy_commit(struct rte_eth_dev *dev,
>  	return ret_val;
>  
>  fail_clear:
> -	if (vf_bw != NULL)
> -		free(vf_bw);
> -	if (tc_bw != NULL)
> -		free(tc_bw);
> +	free(vf_bw);
> +	free(tc_bw);
>  
>  	/* clear all the traffic manager configuration */
>  	if (clear_on_fail) {
> -- 
> 2.53.0
> 


More information about the dev mailing list