]> mj.ucw.cz Git - pciutils.git/commitdiff
Sysfs: fixed sysfs_deref_link()
authorMartin Mares <mj@ucw.cz>
Thu, 12 Jul 2018 11:59:52 +0000 (13:59 +0200)
committerMartin Mares <mj@ucw.cz>
Thu, 12 Jul 2018 11:59:52 +0000 (13:59 +0200)
The function canonicalize_file_name() is GLIBC-specific, use realpath()
instead, which is available also on MUSL libc.

Also, it leaked memory.

lib/sysfs.c

index 1adb50f1c4072e429cde63d9b127eefd9ec05c7c..42c88c69f7c2326eb2ada9b9deaa86c8c857c286 100644 (file)
@@ -133,7 +133,8 @@ sysfs_deref_link(struct pci_dev *d, char *link_name)
   sysfs_obj_name(d, "", path);
   strcat(path, rel_path);
 
-  return canonicalize_file_name(path);
+  // Returns a pointer to malloc'ed memory
+  return realpath(path, NULL);
 }
 
 static int
@@ -329,7 +330,14 @@ sysfs_fill_info(struct pci_dev *d, int flags)
     d->numa_node = sysfs_get_value(d, "numa_node", 0);
 
   if ((flags & PCI_FILL_DT_NODE) && !(d->known_fields & PCI_FILL_DT_NODE))
-    pci_set_property(d, PCI_FILL_DT_NODE, sysfs_deref_link(d, "of_node"));
+    {
+      char *node = sysfs_deref_link(d, "of_node");
+      if (node)
+       {
+         pci_set_property(d, PCI_FILL_DT_NODE, node);
+         free(node);
+       }
+    }
 
   return pci_generic_fill_info(d, flags);
 }