]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
tree-wide: use sd_device_new_from_ifindex/ifname()
authorYu Watanabe <watanabe.yu+github@gmail.com>
Wed, 9 Jun 2021 14:33:50 +0000 (23:33 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Fri, 11 Jun 2021 18:19:08 +0000 (03:19 +0900)
src/libsystemd-network/dhcp-identifier.c
src/network/networkctl.c
src/network/networkd-link.c
src/nspawn/nspawn-network.c

index 953fef19fa407582c3b8ab037a06561aed3fe14b..c3b47ba3c90530b92d1a2a775e15fa10b3dca186 100644 (file)
@@ -167,14 +167,12 @@ int dhcp_identifier_set_iaid(
         const char *name = NULL;
         uint64_t id;
         uint32_t id32;
+        int r;
 
         if (detect_container() <= 0) {
                 /* not in a container, udev will be around */
-                char ifindex_str[1 + DECIMAL_STR_MAX(int)];
-                int r;
 
-                xsprintf(ifindex_str, "n%d", ifindex);
-                if (sd_device_new_from_device_id(&device, ifindex_str) >= 0) {
+                if (sd_device_new_from_ifindex(&device, ifindex) >= 0) {
                         r = sd_device_get_is_initialized(device);
                         if (r < 0)
                                 return r;
index 343dea3ed64d674382175f98cd2cac151a754b85..1c2cb7691a42a965b0bd47ce1aa66529bf44f5d1 100644 (file)
@@ -753,9 +753,7 @@ static int acquire_link_info(sd_bus *bus, sd_netlink *rtnl, char **patterns, Lin
 
                 links[c].needs_freeing = true;
 
-                char devid[2 + DECIMAL_STR_MAX(int)];
-                xsprintf(devid, "n%i", links[c].ifindex);
-                (void) sd_device_new_from_device_id(&links[c].sd_device, devid);
+                (void) sd_device_new_from_ifindex(&links[c].sd_device, links[c].ifindex);
 
                 acquire_ether_link_info(&fd, &links[c]);
                 acquire_wlan_link_info(&links[c]);
index 911fb9b302c2581d1b99f3df261aa5a101ddc1d1..c3af71b48fc531e86886b2623b9a4cb075a2419f 100644 (file)
@@ -1394,7 +1394,6 @@ static int link_initialized(Link *link, sd_device *device) {
 
 static int link_check_initialized(Link *link) {
         _cleanup_(sd_device_unrefp) sd_device *device = NULL;
-        char ifindex_str[2 + DECIMAL_STR_MAX(int)];
         int r;
 
         assert(link);
@@ -1404,8 +1403,7 @@ static int link_check_initialized(Link *link) {
                 return link_initialized_and_synced(link);
 
         /* udev should be around */
-        xsprintf(ifindex_str, "n%d", link->ifindex);
-        r = sd_device_new_from_device_id(&device, ifindex_str);
+        r = sd_device_new_from_ifindex(&device, link->ifindex);
         if (r < 0) {
                 log_link_debug_errno(link, r, "Could not find device, waiting for device initialization: %m");
                 return 0;
index 1f0d27475787ee4e0d2f40fd4d99a072733a5fc8..97e2756658bf3fc7f70627032841dc84ffc277a9 100644 (file)
@@ -467,20 +467,14 @@ int remove_bridge(const char *bridge_name) {
 
 int test_network_interface_initialized(const char *name) {
         _cleanup_(sd_device_unrefp) sd_device *d = NULL;
-        int ifi, r;
-        char ifi_str[2 + DECIMAL_STR_MAX(int)];
+        int r;
 
         if (path_is_read_only_fs("/sys"))
                 return 0;
 
         /* udev should be around. */
 
-        ifi = rtnl_resolve_interface_or_warn(NULL, name);
-        if (ifi < 0)
-                return ifi;
-
-        sprintf(ifi_str, "n%i", ifi);
-        r = sd_device_new_from_device_id(&d, ifi_str);
+        r = sd_device_new_from_ifname(&d, name);
         if (r < 0)
                 return log_error_errno(r, "Failed to get device %s: %m", name);