network,udev: drop unnecessary check whether ifindex is positive

As sd_device_get_ifindex() now returns positive ifindex when it
succeeds.

diff --git a/src/network/networkd-manager.c b/src/network/networkd-manager.c
index 4a621950cafdc7b984dec32fef0644c68f0d7786..f7e9b48609642dc7a55ad3665eb53a5b22b2eae2 100644 (file)
--- a/src/network/networkd-manager.c
+++ b/src/network/networkd-manager.c
@@ -197,8 +197,8 @@ static int manager_udev_process_link(sd_device_monitor *monitor, sd_device *devi
                 return 0;
 
         r = sd_device_get_ifindex(device, &ifindex);
-        if (r < 0 || ifindex <= 0) {
-                log_debug("Ignoring udev ADD event for device with invalid ifindex");
+        if (r < 0) {
+                log_debug_errno(r, "Ignoring udev ADD event for device without ifindex or with invalid ifindex: %m");
                 return 0;
         }
 

diff --git a/src/udev/net/link-config.c b/src/udev/net/link-config.c
index 2d25f753575040e0dc68c7f60d3c7fbf2912fc2b..c39d8791f2a81bc8250a341e031a6e3678e901ed 100644 (file)
--- a/src/udev/net/link-config.c
+++ b/src/udev/net/link-config.c
@@ -412,8 +412,6 @@ int link_config_apply(link_config_ctx *ctx, link_config *config,
         r = sd_device_get_ifindex(device, &ifindex);
         if (r < 0)
                 return log_device_warning_errno(device, r, "Could not find ifindex: %m");
-        if (ifindex <= 0)
-                return log_device_warning_errno(device, EINVAL, "Invalid ifindex '%d'", ifindex);
 
         if (ctx->enable_name_policy && config->name_policy) {
                 NamePolicy *policy;