eal: fix bus name read for removal in multi-process
authorQi Zhang <qi.z.zhang@intel.com>
Mon, 22 Oct 2018 06:15:16 +0000 (14:15 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 22 Oct 2018 10:41:28 +0000 (12:41 +0200)
A crash may appear when removing some PCI devices because
dev->devargs is not always initialized. So use dev->bus instead of
dev->devargs->bus when building devargs string to remove a device.

Fixes: 244d5130719c ("eal: enable hotplug on multi-process")

Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
lib/librte_eal/common/eal_common_dev.c

index 8b0844a..6ac3ee8 100644 (file)
@@ -387,7 +387,7 @@ rte_dev_remove(struct rte_device *dev)
                return -ENOENT;
        }
 
-       ret = build_devargs(dev->devargs->bus->name, dev->name, "", &devargs);
+       ret = build_devargs(dev->bus->name, dev->name, "", &devargs);
        if (ret != 0)
                return ret;