vlib: pci: set pci driver name to none when no driver is loaded 44/22544/2
authorBenoît Ganne <bganne@cisco.com>
Fri, 4 Oct 2019 15:30:21 +0000 (17:30 +0200)
committerDamjan Marion <dmarion@me.com>
Sat, 5 Oct 2019 11:44:50 +0000 (11:44 +0000)
If no Linux PCI driver module is loaded, then the driver_name in the PCI
info struct is NULL. This can triggers crash when checking driver name
eg. in vlib_pci_device_open().
Default to "<NONE>" as driver name, which should never match.

Type: fix

Change-Id: I9e69889a7566467bd8220b92bbbaa72ada957257
Signed-off-by: Benoît Ganne <bganne@cisco.com>
src/vlib/linux/pci.c

index fdb5271..82f5007 100644 (file)
@@ -291,6 +291,8 @@ vlib_pci_get_device_info (vlib_main_t * vm, vlib_pci_addr_t * addr,
   vec_reset_length (f);
   f = format (f, "%v/driver%c", dev_dir_name, 0);
   di->driver_name = clib_sysfs_link_to_name ((char *) f);
+  if (!di->driver_name)
+    di->driver_name = format (0, "<NONE>%c", 0);
 
   di->iommu_group = -1;
   if (lvm->container_fd != -1)