]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
network/route: trivial cosmetic changes 30949/head
authorYu Watanabe <watanabe.yu+github@gmail.com>
Sun, 14 Jan 2024 05:18:16 +0000 (14:18 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Tue, 16 Jan 2024 02:45:14 +0000 (11:45 +0900)
- downgrade log level,
- add short comment for boolean argument.

src/network/networkd-route.c

index 4b8bdb39f32f861f5991eaba1882592955f90fc8..b0406b75229639a22c7154ca296ac71c38d0c20f 100644 (file)
@@ -672,11 +672,11 @@ int route_remove(Route *route) {
 
         r = sd_rtnl_message_new_route(manager->rtnl, &m, RTM_DELROUTE, route->family, route->protocol);
         if (r < 0)
-                return log_link_error_errno(link, r, "Could not create netlink message: %m");
+                return log_link_warning_errno(link, r, "Could not create netlink message: %m");
 
         r = route_set_netlink_message(route, m, link);
         if (r < 0)
-                return log_error_errno(r, "Could not fill netlink message: %m");
+                return log_link_warning_errno(link, r, "Could not fill netlink message: %m");
 
         if (route->family == AF_INET && route->nexthop_id > 0 && route->type == RTN_BLACKHOLE)
                 /* When IPv4 route has nexthop id and the nexthop type is blackhole, even though kernel
@@ -696,7 +696,7 @@ int route_remove(Route *route) {
         r = netlink_call_async(manager->rtnl, NULL, m, route_remove_handler,
                                link ? link_netlink_destroy_callback : NULL, link);
         if (r < 0)
-                return log_link_error_errno(link, r, "Could not send netlink message: %m");
+                return log_link_warning_errno(link, r, "Could not send netlink message: %m");
 
         link_ref(link);
 
@@ -1043,7 +1043,7 @@ static int route_is_ready_to_configure(const Route *route, Link *link) {
         assert(route);
         assert(link);
 
-        if (!link_is_ready_to_configure(link, false))
+        if (!link_is_ready_to_configure(link, /* allow_unmanaged = */ false))
                 return false;
 
         if (set_size(link->routes) >= routes_max())