From 09d04a74c1d28fff15acbf19c917996dd6d2dd8d Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Sun, 14 Oct 2018 01:04:29 +0900 Subject: [PATCH] udev: replace udev_device by sd_device in udev-builtin-net_id.c --- src/udev/udev-builtin-net_id.c | 447 +++++++++++++++++++++++------------------ 1 file changed, 252 insertions(+), 195 deletions(-) diff --git a/src/udev/udev-builtin-net_id.c b/src/udev/udev-builtin-net_id.c index e154661..7e6b5a8 100644 --- a/src/udev/udev-builtin-net_id.c +++ b/src/udev/udev-builtin-net_id.c @@ -133,7 +133,7 @@ struct netnames { uint8_t mac[6]; bool mac_valid; - struct udev_device *pcidev; + sd_device *pcidev; char pci_slot[IFNAMSIZ]; char pci_path[IFNAMSIZ]; char pci_onboard[IFNAMSIZ]; @@ -147,94 +147,111 @@ struct netnames { }; struct virtfn_info { - struct udev_device *physfn_pcidev; + sd_device *physfn_pcidev; char suffix[IFNAMSIZ]; }; /* skip intermediate virtio devices */ -static struct udev_device *skip_virtio(struct udev_device *dev) { - struct udev_device *parent = dev; +static sd_device *skip_virtio(sd_device *dev) { + sd_device *parent; /* there can only ever be one virtio bus per parent device, so we can - safely ignore any virtio buses. see - */ - while (parent && streq_ptr("virtio", udev_device_get_subsystem(parent))) - parent = udev_device_get_parent(parent); + * safely ignore any virtio buses. see + * http://lists.linuxfoundation.org/pipermail/virtualization/2015-August/030331.html */ + for (parent = dev; parent; ) { + const char *subsystem; + + if (sd_device_get_subsystem(parent, &subsystem) < 0) + break; + + if (!streq(subsystem, "virtio")) + break; + + if (sd_device_get_parent(parent, &parent) < 0) + return NULL; + } + return parent; } -static int get_virtfn_info(struct udev_device *dev, struct netnames *names, struct virtfn_info *vf_info) { - const char *physfn_link_file; +static int get_virtfn_info(sd_device *dev, struct netnames *names, struct virtfn_info *ret) { + _cleanup_(sd_device_unrefp) sd_device *physfn_pcidev = NULL; + const char *physfn_link_file, *syspath; _cleanup_free_ char *physfn_pci_syspath = NULL; _cleanup_free_ char *virtfn_pci_syspath = NULL; struct dirent *dent; _cleanup_closedir_ DIR *dir = NULL; - struct virtfn_info vf_info_local = {}; + char suffix[IFNAMSIZ]; int r; + assert(dev); + assert(names); + assert(ret); + + r = sd_device_get_syspath(names->pcidev, &syspath); + if (r < 0) + return r; + /* Check if this is a virtual function. */ - physfn_link_file = strjoina(udev_device_get_syspath(names->pcidev), "/physfn"); + physfn_link_file = strjoina(syspath, "/physfn"); r = chase_symlinks(physfn_link_file, NULL, 0, &physfn_pci_syspath); if (r < 0) return r; /* Get physical function's pci device. */ - vf_info_local.physfn_pcidev = udev_device_new_from_syspath(NULL, physfn_pci_syspath); - if (!vf_info_local.physfn_pcidev) - return -ENOENT; + r = sd_device_new_from_syspath(&physfn_pcidev, physfn_pci_syspath); + if (r < 0) + return r; /* Find the virtual function number by finding the right virtfn link. */ dir = opendir(physfn_pci_syspath); - if (!dir) { - r = -errno; - goto out_unref; - } + if (!dir) + return -errno; + FOREACH_DIRENT_ALL(dent, dir, break) { _cleanup_free_ char *virtfn_link_file = NULL; + if (!startswith(dent->d_name, "virtfn")) continue; + virtfn_link_file = strjoin(physfn_pci_syspath, "/", dent->d_name); - if (!virtfn_link_file) { - r = -ENOMEM; - goto out_unref; - } + if (!virtfn_link_file) + return -ENOMEM; + if (chase_symlinks(virtfn_link_file, NULL, 0, &virtfn_pci_syspath) < 0) continue; - if (streq(udev_device_get_syspath(names->pcidev), virtfn_pci_syspath)) { - if (!snprintf_ok(vf_info_local.suffix, sizeof(vf_info_local.suffix), "v%s", &dent->d_name[6])) { - r = -ENOENT; - goto out_unref; - } + + if (streq(syspath, virtfn_pci_syspath)) { + if (!snprintf_ok(suffix, sizeof(suffix), "v%s", &dent->d_name[6])) + return -ENOENT; + break; } } - if (isempty(vf_info_local.suffix)) { - r = -ENOENT; - goto out_unref; - } - *vf_info = vf_info_local; - return 0; + if (isempty(suffix)) + return -ENOENT; + + ret->physfn_pcidev = TAKE_PTR(physfn_pcidev); + strncpy(ret->suffix, suffix, sizeof(ret->suffix)); -out_unref: - udev_device_unref(vf_info_local.physfn_pcidev); - return r; + return 0; } /* retrieve on-board index number and label from firmware */ -static int dev_pci_onboard(struct udev_device *dev, struct netnames *names) { +static int dev_pci_onboard(sd_device *dev, struct netnames *names) { unsigned long idx, dev_port = 0; - const char *attr, *port_name; + const char *attr, *port_name = NULL; size_t l; char *s; int r; /* ACPI _DSM — device specific method for naming a PCI or PCI Express device */ - attr = udev_device_get_sysattr_value(names->pcidev, "acpi_index"); - /* SMBIOS type 41 — Onboard Devices Extended Information */ - if (!attr) - attr = udev_device_get_sysattr_value(names->pcidev, "index"); - if (!attr) - return -ENOENT; + if (sd_device_get_sysattr_value(names->pcidev, "acpi_index", &attr) < 0) { + /* SMBIOS type 41 — Onboard Devices Extended Information */ + r = sd_device_get_sysattr_value(names->pcidev, "index", &attr); + if (r < 0) + return r; + } r = safe_atolu(attr, &idx); if (r < 0) @@ -248,12 +265,11 @@ static int dev_pci_onboard(struct udev_device *dev, struct netnames *names) { return -ENOENT; /* kernel provided port index for multiple ports on a single PCI function */ - attr = udev_device_get_sysattr_value(dev, "dev_port"); - if (attr) + if (sd_device_get_sysattr_value(dev, "dev_port", &attr) >= 0) dev_port = strtoul(attr, NULL, 10); /* kernel provided front panel port name for multiple port PCI device */ - port_name = udev_device_get_sysattr_value(dev, "phys_port_name"); + (void) sd_device_get_sysattr_value(dev, "phys_port_name", &port_name); s = names->pci_onboard; l = sizeof(names->pci_onboard); @@ -265,18 +281,22 @@ static int dev_pci_onboard(struct udev_device *dev, struct netnames *names) { if (l == 0) names->pci_onboard[0] = '\0'; - names->pci_onboard_label = udev_device_get_sysattr_value(names->pcidev, "label"); + if (sd_device_get_sysattr_value(names->pcidev, "label", &names->pci_onboard_label) < 0) + names->pci_onboard_label = NULL; return 0; } /* read the 256 bytes PCI configuration space to check the multi-function bit */ -static bool is_pci_multifunction(struct udev_device *dev) { +static bool is_pci_multifunction(sd_device *dev) { _cleanup_close_ int fd = -1; - const char *filename; + const char *filename, *syspath; uint8_t config[64]; - filename = strjoina(udev_device_get_syspath(dev), "/config"); + if (sd_device_get_syspath(dev, &syspath) < 0) + return false; + + filename = strjoina(syspath, "/config"); fd = open(filename, O_RDONLY | O_CLOEXEC); if (fd < 0) return false; @@ -284,30 +304,38 @@ static bool is_pci_multifunction(struct udev_device *dev) { return false; /* bit 0-6 header type, bit 7 multi/single function device */ - if ((config[PCI_HEADER_TYPE] & 0x80) != 0) - return true; - - return false; + return config[PCI_HEADER_TYPE] & 0x80; } -static bool is_pci_ari_enabled(struct udev_device *dev) { - return streq_ptr(udev_device_get_sysattr_value(dev, "ari_enabled"), "1"); +static bool is_pci_ari_enabled(sd_device *dev) { + const char *a; + + if (sd_device_get_sysattr_value(dev, "ari_enabled", &a) < 0) + return false; + + return streq(a, "1"); } -static int dev_pci_slot(struct udev_device *dev, struct netnames *names) { +static int dev_pci_slot(sd_device *dev, struct netnames *names) { unsigned long dev_port = 0; unsigned domain, bus, slot, func, hotplug_slot = 0; size_t l; char *s; - const char *attr, *port_name; - _cleanup_(udev_device_unrefp) struct udev_device *pci = NULL; - struct udev_device *hotplug_slot_dev; + const char *sysname, *attr, *port_name = NULL, *syspath; + _cleanup_(sd_device_unrefp) sd_device *pci = NULL; + sd_device *hotplug_slot_dev; char slots[PATH_MAX]; _cleanup_closedir_ DIR *dir = NULL; struct dirent *dent; + int r; - if (sscanf(udev_device_get_sysname(names->pcidev), "%x:%x:%x.%u", &domain, &bus, &slot, &func) != 4) + r = sd_device_get_sysname(names->pcidev, &sysname); + if (r < 0) + return r; + + if (sscanf(sysname, "%x:%x:%x.%u", &domain, &bus, &slot, &func) != 4) return -ENOENT; + if (is_pci_ari_enabled(names->pcidev)) /* ARI devices support up to 256 functions on a single device ("slot"), and interpret the * traditional 5-bit slot and 3-bit function number as a single 8-bit function number, @@ -315,27 +343,24 @@ static int dev_pci_slot(struct udev_device *dev, struct netnames *names) { func += slot * 8; /* kernel provided port index for multiple ports on a single PCI function */ - attr = udev_device_get_sysattr_value(dev, "dev_port"); - if (attr) { + if (sd_device_get_sysattr_value(dev, "dev_port", &attr) >= 0) { dev_port = strtoul(attr, NULL, 10); /* With older kernels IP-over-InfiniBand network interfaces sometimes erroneously * provide the port number in the 'dev_id' sysfs attribute instead of 'dev_port', * which thus stays initialized as 0. */ - if (dev_port == 0) { - attr = udev_device_get_sysattr_value(dev, "type"); - if (attr) { - unsigned long type = strtoul(attr, NULL, 10); - if (type == ARPHRD_INFINIBAND) { - attr = udev_device_get_sysattr_value(dev, "dev_id"); - if (attr) - dev_port = strtoul(attr, NULL, 16); - } - } + if (dev_port == 0 && + sd_device_get_sysattr_value(dev, "type", &attr) >= 0) { + unsigned long type; + + type = strtoul(attr, NULL, 10); + if (type == ARPHRD_INFINIBAND && + sd_device_get_sysattr_value(dev, "dev_id", &attr) >= 0) + dev_port = strtoul(attr, NULL, 16); } } /* kernel provided front panel port name for multiple port PCI device */ - port_name = udev_device_get_sysattr_value(dev, "phys_port_name"); + (void) sd_device_get_sysattr_value(dev, "phys_port_name", &port_name); /* compose a name based on the raw kernel's PCI bus, slot numbers */ s = names->pci_path; @@ -353,11 +378,14 @@ static int dev_pci_slot(struct udev_device *dev, struct netnames *names) { names->pci_path[0] = '\0'; /* ACPI _SUN — slot user number */ - pci = udev_device_new_from_subsystem_sysname(NULL, "subsystem", "pci"); - if (!pci) - return -ENOENT; + r = sd_device_new_from_subsystem_sysname(&pci, "subsystem", "pci"); + if (r < 0) + return r; - if (!snprintf_ok(slots, sizeof slots, "%s/slots", udev_device_get_syspath(pci))) + r = sd_device_get_syspath(pci, &syspath); + if (r < 0) + return r; + if (!snprintf_ok(slots, sizeof slots, "%s/slots", syspath)) return -ENAMETOOLONG; dir = opendir(slots); @@ -366,31 +394,33 @@ static int dev_pci_slot(struct udev_device *dev, struct netnames *names) { hotplug_slot_dev = names->pcidev; while (hotplug_slot_dev) { + if (sd_device_get_sysname(hotplug_slot_dev, &sysname) < 0) + continue; + FOREACH_DIRENT_ALL(dent, dir, break) { unsigned i; - int r; char str[PATH_MAX]; _cleanup_free_ char *address = NULL; if (dent->d_name[0] == '.') continue; r = safe_atou_full(dent->d_name, 10, &i); - if (i < 1 || r < 0) + if (r < 0 || i <= 0) continue; + /* match slot address with device by stripping the function */ if (snprintf_ok(str, sizeof str, "%s/%s/address", slots, dent->d_name) && - read_one_line_file(str, &address) >= 0) - /* match slot address with device by stripping the function */ - if (startswith(udev_device_get_sysname(hotplug_slot_dev), address)) - hotplug_slot = i; - - if (hotplug_slot > 0) + read_one_line_file(str, &address) >= 0 && + startswith(sysname, address)) { + hotplug_slot = i; break; + } } if (hotplug_slot > 0) break; + if (sd_device_get_parent_with_subsystem_devtype(hotplug_slot_dev, "pci", NULL, &hotplug_slot_dev) < 0) + break; rewinddir(dir); - hotplug_slot_dev = udev_device_get_parent_with_subsystem_devtype(hotplug_slot_dev, "pci", NULL); } if (hotplug_slot > 0) { @@ -412,24 +442,30 @@ static int dev_pci_slot(struct udev_device *dev, struct netnames *names) { return 0; } -static int names_vio(struct udev_device *dev, struct netnames *names) { - struct udev_device *parent; +static int names_vio(sd_device *dev, struct netnames *names) { + sd_device *parent; unsigned busid, slotid, ethid; - const char *syspath; + const char *syspath, *subsystem; + int r; /* check if our direct parent is a VIO device with no other bus in-between */ - parent = udev_device_get_parent(dev); - if (!parent) - return -ENOENT; + r = sd_device_get_parent(dev, &parent); + if (r < 0) + return r; - if (!streq_ptr("vio", udev_device_get_subsystem(parent))) - return -ENOENT; + r = sd_device_get_subsystem(parent, &subsystem); + if (r < 0) + return r; + if (!streq("vio", subsystem)) + return -ENOENT; /* The devices' $DEVPATH number is tied to (virtual) hardware (slot id * selected in the HMC), thus this provides a reliable naming (e.g. * "/devices/vio/30000002/net/eth1"); we ignore the bus number, as * there should only ever be one bus, and then remove leading zeros. */ - syspath = udev_device_get_syspath(dev); + r = sd_device_get_syspath(dev, &syspath); + if (r < 0) + return r; if (sscanf(syspath, "/sys/devices/vio/%4x%4x/net/eth%u", &busid, &slotid, ðid) != 3) return -EINVAL; @@ -443,21 +479,28 @@ static int names_vio(struct udev_device *dev, struct netnames *names) { #define _PLATFORM_PATTERN4 "/sys/devices/platform/%4s%4x:%2x/net/eth%u" #define _PLATFORM_PATTERN3 "/sys/devices/platform/%3s%4x:%2x/net/eth%u" -static int names_platform(struct udev_device *dev, struct netnames *names, bool test) { - struct udev_device *parent; +static int names_platform(sd_device *dev, struct netnames *names, bool test) { + sd_device *parent; char vendor[5]; unsigned model, instance, ethid; - const char *syspath, *pattern, *validchars; + const char *syspath, *pattern, *validchars, *subsystem; + int r; /* check if our direct parent is a platform device with no other bus in-between */ - parent = udev_device_get_parent(dev); - if (!parent) - return -ENOENT; + r = sd_device_get_parent(dev, &parent); + if (r < 0) + return r; + + r = sd_device_get_subsystem(parent, &subsystem); + if (r < 0) + return r; - if (!streq_ptr("platform", udev_device_get_subsystem(parent))) + if (!streq("platform", subsystem)) return -ENOENT; - syspath = udev_device_get_syspath(dev); + r = sd_device_get_syspath(dev, &syspath); + if (r < 0) + return r; /* syspath is too short, to have a valid ACPI instance */ if (strlen(syspath) < sizeof _PLATFORM_TEST) @@ -493,15 +536,19 @@ static int names_platform(struct udev_device *dev, struct netnames *names, bool return 0; } -static int names_pci(struct udev_device *dev, struct netnames *names) { - struct udev_device *parent; +static int names_pci(sd_device *dev, struct netnames *names) { + sd_device *parent; struct netnames vf_names = {}; struct virtfn_info vf_info = {}; + const char *subsystem; + int r; assert(dev); assert(names); - parent = udev_device_get_parent(dev); + r = sd_device_get_parent(dev, &parent); + if (r < 0) + return r; /* skip virtio subsystem if present */ parent = skip_virtio(parent); @@ -509,13 +556,14 @@ static int names_pci(struct udev_device *dev, struct netnames *names) { return -ENOENT; /* check if our direct parent is a PCI device with no other bus in-between */ - if (streq_ptr("pci", udev_device_get_subsystem(parent))) { + if (sd_device_get_subsystem(parent, &subsystem) >= 0 && + streq("pci", subsystem)) { names->type = NET_PCI; names->pcidev = parent; } else { - names->pcidev = udev_device_get_parent_with_subsystem_devtype(dev, "pci", NULL); - if (!names->pcidev) - return -ENOENT; + r = sd_device_get_parent_with_subsystem_devtype(dev, "pci", NULL, &names->pcidev); + if (r < 0) + return r; } if (get_virtfn_info(dev, names, &vf_info) >= 0) { @@ -535,32 +583,35 @@ static int names_pci(struct udev_device *dev, struct netnames *names) { if (strlen(vf_names.pci_path) + strlen(vf_info.suffix) < sizeof(names->pci_path)) strscpyl(names->pci_path, sizeof(names->pci_path), vf_names.pci_path, vf_info.suffix, NULL); - udev_device_unref(vf_info.physfn_pcidev); + sd_device_unref(vf_info.physfn_pcidev); } else { dev_pci_onboard(dev, names); dev_pci_slot(dev, names); } + return 0; } -static int names_usb(struct udev_device *dev, struct netnames *names) { - struct udev_device *usbdev; - char name[256]; - char *ports; - char *config; - char *interf; +static int names_usb(sd_device *dev, struct netnames *names) { + sd_device *usbdev; + char name[256], *ports, *config, *interf, *s; + const char *sysname; size_t l; - char *s; + int r; assert(dev); assert(names); - usbdev = udev_device_get_parent_with_subsystem_devtype(dev, "usb", "usb_interface"); - if (!usbdev) - return -ENOENT; + r = sd_device_get_parent_with_subsystem_devtype(dev, "usb", "usb_interface", &usbdev); + if (r < 0) + return r; + + r = sd_device_get_sysname(usbdev, &sysname); + if (r < 0) + return r; /* get USB port number chain, configuration, interface */ - strscpy(name, sizeof(name), udev_device_get_sysname(usbdev)); + strscpy(name, sizeof(name), sysname); s = strchr(name, '-'); if (!s) return -EINVAL; @@ -599,19 +650,25 @@ static int names_usb(struct udev_device *dev, struct netnames *names) { return 0; } -static int names_bcma(struct udev_device *dev, struct netnames *names) { - struct udev_device *bcmadev; +static int names_bcma(sd_device *dev, struct netnames *names) { + sd_device *bcmadev; unsigned core; + const char *sysname; + int r; assert(dev); assert(names); - bcmadev = udev_device_get_parent_with_subsystem_devtype(dev, "bcma", NULL); - if (!bcmadev) - return -ENOENT; + r = sd_device_get_parent_with_subsystem_devtype(dev, "bcma", NULL, &bcmadev); + if (r < 0) + return r; + + r = sd_device_get_sysname(bcmadev, &sysname); + if (r < 0) + return r; /* bus num:core num */ - if (sscanf(udev_device_get_sysname(bcmadev), "bcma%*u:%u", &core) != 1) + if (sscanf(sysname, "bcma%*u:%u", &core) != 1) return -EINVAL; /* suppress the common core == 0 */ if (core > 0) @@ -621,34 +678,41 @@ static int names_bcma(struct udev_device *dev, struct netnames *names) { return 0; } -static int names_ccw(struct udev_device *dev, struct netnames *names) { - struct udev_device *cdev; +static int names_ccw(sd_device *dev, struct netnames *names) { + sd_device *cdev; const char *bus_id, *subsys; size_t bus_id_len; size_t bus_id_start; + int r; assert(dev); assert(names); /* Retrieve the associated CCW device */ - cdev = udev_device_get_parent(dev); + r = sd_device_get_parent(dev, &cdev); + if (r < 0) + return r; + /* skip virtio subsystem if present */ cdev = skip_virtio(cdev); if (!cdev) return -ENOENT; + r = sd_device_get_subsystem(cdev, &subsys); + if (r < 0) + return r; + /* Network devices are either single or grouped CCW devices */ - subsys = udev_device_get_subsystem(cdev); - if (!STRPTR_IN_SET(subsys, "ccwgroup", "ccw")) + if (!STR_IN_SET(subsys, "ccwgroup", "ccw")) return -ENOENT; /* Retrieve bus-ID of the CCW device. The bus-ID uniquely * identifies the network device on the Linux on System z channel * subsystem. Note that the bus-ID contains lowercase characters. */ - bus_id = udev_device_get_sysname(cdev); - if (!bus_id) - return -ENOENT; + r = sd_device_get_sysname(cdev, &bus_id); + if (r < 0) + return r; /* Check the length of the bus-ID. Rely on that the kernel provides * a correct bus-ID; alternatively, improve this check and parse and @@ -673,17 +737,19 @@ static int names_ccw(struct udev_device *dev, struct netnames *names) { return 0; } -static int names_mac(struct udev_device *dev, struct netnames *names) { +static int names_mac(sd_device *dev, struct netnames *names) { const char *s; unsigned long i; unsigned a1, a2, a3, a4, a5, a6; + int r; /* Some kinds of devices tend to have hardware addresses * that are impossible to use in an iface name. */ - s = udev_device_get_sysattr_value(dev, "type"); - if (!s) - return EXIT_FAILURE; + r = sd_device_get_sysattr_value(dev, "type", &s); + if (r < 0) + return r; + i = strtoul(s, NULL, 0); switch (i) { /* The persistent part of a hardware address of an InfiniBand NIC @@ -696,16 +762,16 @@ static int names_mac(struct udev_device *dev, struct netnames *names) { } /* check for NET_ADDR_PERM, skip random MAC addresses */ - s = udev_device_get_sysattr_value(dev, "addr_assign_type"); - if (!s) - return EXIT_FAILURE; + r = sd_device_get_sysattr_value(dev, "addr_assign_type", &s); + if (r < 0) + return r; i = strtoul(s, NULL, 0); if (i != 0) return 0; - s = udev_device_get_sysattr_value(dev, "address"); - if (!s) - return -ENOENT; + r = sd_device_get_sysattr_value(dev, "address", &s); + if (r < 0) + return r; if (sscanf(s, "%x:%x:%x:%x:%x:%x", &a1, &a2, &a3, &a4, &a5, &a6) != 6) return -EINVAL; @@ -724,7 +790,7 @@ static int names_mac(struct udev_device *dev, struct netnames *names) { } /* IEEE Organizationally Unique Identifier vendor string */ -static int ieee_oui(struct udev_device *dev, struct netnames *names, bool test) { +static int ieee_oui(sd_device *dev, struct netnames *names, bool test) { char str[32]; if (!names->mac_valid) @@ -735,25 +801,22 @@ static int ieee_oui(struct udev_device *dev, struct netnames *names, bool test) xsprintf(str, "OUI:%02X%02X%02X%02X%02X%02X", names->mac[0], names->mac[1], names->mac[2], names->mac[3], names->mac[4], names->mac[5]); - udev_builtin_hwdb_lookup(dev->device, NULL, str, NULL, test); + udev_builtin_hwdb_lookup(dev, NULL, str, NULL, test); return 0; } -static int builtin_net_id(struct udev_device *dev, int argc, char *argv[], bool test) { - const char *s; - const char *p; - unsigned long i; - const char *devtype; - const char *prefix = "en"; +static int builtin_net_id(struct udev_device *_dev, int argc, char *argv[], bool test) { + const char *s, *p, *devtype, *prefix = "en"; struct netnames names = {}; - int err; + unsigned long i; + int r; + sd_device *dev = _dev->device; - /* handle only ARPHRD_ETHER, ARPHRD_SLIP - * and ARPHRD_INFINIBAND devices - */ - s = udev_device_get_sysattr_value(dev, "type"); - if (!s) + /* handle only ARPHRD_ETHER, ARPHRD_SLIP and ARPHRD_INFINIBAND devices */ + r = sd_device_get_sysattr_value(dev, "type", &s); + if (r < 0) return EXIT_FAILURE; + i = strtoul(s, NULL, 0); switch (i) { case ARPHRD_ETHER: @@ -770,68 +833,63 @@ static int builtin_net_id(struct udev_device *dev, int argc, char *argv[], bool } /* skip stacked devices, like VLANs, ... */ - s = udev_device_get_sysattr_value(dev, "ifindex"); - if (!s) + r = sd_device_get_sysattr_value(dev, "ifindex", &s); + if (r < 0) return EXIT_FAILURE; - p = udev_device_get_sysattr_value(dev, "iflink"); - if (!p) + r = sd_device_get_sysattr_value(dev, "iflink", &p); + if (r < 0) return EXIT_FAILURE; if (!streq(s, p)) return 0; - devtype = udev_device_get_devtype(dev); - if (devtype) { + if (sd_device_get_devtype(dev, &devtype) >= 0) { if (streq("wlan", devtype)) prefix = "wl"; else if (streq("wwan", devtype)) prefix = "ww"; } - err = names_mac(dev, &names); - if (err >= 0 && names.mac_valid) { + r = names_mac(dev, &names); + if (r >= 0 && names.mac_valid) { char str[IFNAMSIZ]; xsprintf(str, "%sx%02x%02x%02x%02x%02x%02x", prefix, names.mac[0], names.mac[1], names.mac[2], names.mac[3], names.mac[4], names.mac[5]); - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_MAC", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_MAC", str); ieee_oui(dev, &names, test); } /* get path names for Linux on System z network devices */ - err = names_ccw(dev, &names); - if (err >= 0 && names.type == NET_CCW) { + if (names_ccw(dev, &names) >= 0 && names.type == NET_CCW) { char str[IFNAMSIZ]; if (snprintf_ok(str, sizeof str, "%s%s", prefix, names.ccw_busid)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_PATH", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_PATH", str); goto out; } /* get ibmveth/ibmvnic slot-based names. */ - err = names_vio(dev, &names); - if (err >= 0 && names.type == NET_VIO) { + if (names_vio(dev, &names) >= 0 && names.type == NET_VIO) { char str[IFNAMSIZ]; if (snprintf_ok(str, sizeof str, "%s%s", prefix, names.vio_slot)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_SLOT", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_SLOT", str); goto out; } /* get ACPI path names for ARM64 platform devices */ - err = names_platform(dev, &names, test); - if (err >= 0 && names.type == NET_PLATFORM) { + if (names_platform(dev, &names, test) >= 0 && names.type == NET_PLATFORM) { char str[IFNAMSIZ]; if (snprintf_ok(str, sizeof str, "%s%s", prefix, names.platform_path)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_PATH", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_PATH", str); goto out; } /* get PCI based path names, we compose only PCI based paths */ - err = names_pci(dev, &names); - if (err < 0) + if (names_pci(dev, &names) < 0) goto out; /* plain PCI device */ @@ -840,51 +898,50 @@ static int builtin_net_id(struct udev_device *dev, int argc, char *argv[], bool if (names.pci_onboard[0] && snprintf_ok(str, sizeof str, "%s%s", prefix, names.pci_onboard)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_ONBOARD", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_ONBOARD", str); if (names.pci_onboard_label && snprintf_ok(str, sizeof str, "%s%s", prefix, names.pci_onboard_label)) - udev_builtin_add_property(dev->device, test, "ID_NET_LABEL_ONBOARD", str); + udev_builtin_add_property(dev, test, "ID_NET_LABEL_ONBOARD", str); if (names.pci_path[0] && snprintf_ok(str, sizeof str, "%s%s", prefix, names.pci_path)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_PATH", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_PATH", str); if (names.pci_slot[0] && snprintf_ok(str, sizeof str, "%s%s", prefix, names.pci_slot)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_SLOT", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_SLOT", str); goto out; } /* USB device */ - err = names_usb(dev, &names); - if (err >= 0 && names.type == NET_USB) { + if (names_usb(dev, &names) >= 0 && names.type == NET_USB) { char str[IFNAMSIZ]; if (names.pci_path[0] && snprintf_ok(str, sizeof str, "%s%s%s", prefix, names.pci_path, names.usb_ports)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_PATH", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_PATH", str); if (names.pci_slot[0] && snprintf_ok(str, sizeof str, "%s%s%s", prefix, names.pci_slot, names.usb_ports)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_SLOT", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_SLOT", str); goto out; } /* Broadcom bus */ - err = names_bcma(dev, &names); - if (err >= 0 && names.type == NET_BCMA) { + if (names_bcma(dev, &names) >= 0 && names.type == NET_BCMA) { char str[IFNAMSIZ]; if (names.pci_path[0] && snprintf_ok(str, sizeof str, "%s%s%s", prefix, names.pci_path, names.bcma_core)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_PATH", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_PATH", str); if (names.pci_slot[0] && snprintf(str, sizeof str, "%s%s%s", prefix, names.pci_slot, names.bcma_core)) - udev_builtin_add_property(dev->device, test, "ID_NET_NAME_SLOT", str); + udev_builtin_add_property(dev, test, "ID_NET_NAME_SLOT", str); goto out; } + out: return EXIT_SUCCESS; } -- 2.7.4