No need to check for null before calling free. Found by coccinelle script null_free.
Signed-off-by: Stephen Hemminger <[email protected]> --- drivers/net/intel/ice/ice_dcf_sched.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) 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
