This patch fixes rte_eal_hotplug_add without checking return value issue

Signed-off-by: Rosen Xu <rosen...@intel.com>
Fixes: ef1e8ede3da5 ("raw/ifpga: add Intel FPGA bus rawdev driver")
Cc: rosen...@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);
 end:
        if (kvlist)
@@ -591,7 +592,7 @@
        if (name)
                free(name);
 
-       return 0;
+       return ret;
 }
 
 static int
-- 
1.8.3.1

Reply via email to