Hi,
> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Thursday, February 10, 2022 3:18
> To: dev@dpdk.org
> Cc: Stephen Hemminger <step...@networkplumber.org>; Hemant Agrawal
> <hemant.agra...@nxp.com>; Jakub Palider <jpali...@marvell.com>;
> Tomasz Duszynski <tduszyn...@marvell.com>; Nipun Gupta
> <nipun.gu...@nxp.com>; Xu, Rosen <rosen...@intel.com>; Zhang, Tianfei
> <tianfei.zh...@intel.com>
> Subject: [PATCH v3 32/36] raw/*: remove unnecessary NULL checks
>
> Remove redundant NULL pointer checks before free functions found by
> nullfree.cocci
>
> Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
> Acked-by: Hemant Agrawal <hemant.agra...@nxp.com>
> ---
> drivers/raw/cnxk_bphy/cnxk_bphy_cgx.c | 3 +--
> drivers/raw/dpaa2_qdma/dpaa2_qdma.c | 15 +++++----------
> drivers/raw/ifpga/ifpga_rawdev.c | 9 +++------
> drivers/raw/skeleton/skeleton_rawdev_test.c | 6 ++----
> 4 files changed, 11 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/raw/ifpga/ifpga_rawdev.c
> b/drivers/raw/ifpga/ifpga_rawdev.c
> index fdf3c23d2c5f..b73512d3ada5 100644
> --- a/drivers/raw/ifpga/ifpga_rawdev.c
> +++ b/drivers/raw/ifpga/ifpga_rawdev.c
> @@ -836,8 +836,7 @@ rte_fpga_do_pr(struct rte_rawdev *rawdev, int
> port_id,
> }
>
> free_buffer:
> - if (buf_to_free)
> - rte_free(buf_to_free);
> + rte_free(buf_to_free);
> close_fd:
> close(file_fd);
> file_fd = 0;
> @@ -1739,10 +1738,8 @@ ifpga_cfg_probe(struct rte_vdev_device *dev)
> ret = rte_eal_hotplug_add(RTE_STR(IFPGA_BUS_NAME),
> dev_name, devargs->args);
> end:
> - if (kvlist)
> - rte_kvargs_free(kvlist);
> - if (name)
> - free(name);
> + rte_kvargs_free(kvlist);
> + free(name);
>
> return ret;
> }
Reviewed-by Rosen Xu <rosen...@intel.com>
