[dpdk-dev] [PATCH] drivers/raw/ifpga_rawdev: fix coverity issue 323508
Shreyansh Jain
shreyansh.jain at nxp.com
Tue Oct 23 09:09:52 CEST 2018
Besides the comment I sent before about 'Fixes' before sign-off, a
single trivial comment inline ...
On Tuesday 23 October 2018 07:20 AM, Rosen Xu wrote:
> This patch fixes rte_eal_hotplug_add without checking return value issue
>
> Signed-off-by: Rosen Xu <rosen.xu at intel.com>
> Fixes: ef1e8ede3da5 ("raw/ifpga: add Intel FPGA bus rawdev driver")
> Cc: rosen.xu at intel.com
> ---
> drivers/raw/ifpga_rawdev/ifpga_rawdev.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/raw/ifpga_rawdev/ifpga_rawdev.c b/drivers/raw/ifpga_rawdev/ifpga_rawdev.c
> index 3fed057..32e318f 100644
> --- a/drivers/raw/ifpga_rawdev/ifpga_rawdev.c
> +++ b/drivers/raw/ifpga_rawdev/ifpga_rawdev.c
> @@ -542,6 +542,7 @@
> int port;
> char *name = NULL;
> char dev_name[RTE_RAWDEV_NAME_MAX_LEN];
> + int ret = -1;
>
> devargs = dev->device.devargs;
>
> @@ -583,7 +584,7 @@
> snprintf(dev_name, RTE_RAWDEV_NAME_MAX_LEN, "%d|%s",
> port, name);
>
> - rte_eal_hotplug_add(RTE_STR(IFPGA_BUS_NAME),
> + ret = rte_eal_hotplug_add(RTE_STR(IFPGA_BUS_NAME),
> dev_name, devargs->args);
Ideally, the function argument spreading on next line should start
underneath the previous arguments - something like:
ret = rte_eal_hotplug_add(RTE_STR(IFPGA_BUS_NAME),
dev_name, devargs->args);
But, in this file this is not being done at multiple places (for
example, the snprintf in this code snippet). So, either you can ignore
this comment, or fix it for just this change.
> end:
> if (kvlist)
> @@ -591,7 +592,7 @@
> if (name)
> free(name);
>
> - return 0;
> + return ret;
> }
>
> static int
>
Otherwise, the patch is simple enough.
Acked-by: Shreyansh Jain <shreyansh.jain at nxp.com>
More information about the dev
mailing list