mirrors/systemd
0
0
Fork 0
mirror of https://github.com/systemd/systemd.git synced 2024-11-23 10:13:34 +08:00
Code Issues Packages Projects Releases Wiki Activity

network/ndisc: several cleanups for ndisc_remove_route()

Browse Source 
- drop unnecessary call of ndisc_set_route_priority() at the beginning,
  as it is called later in the loop below,
- use RET_GATHER() and remove all possible routes even if failed.
This commit is contained in:
Yu Watanabe 2024-11-12 02:06:21 +09:00
parent 2437ebee20
commit 42d9660f10
1 changed files with 4 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/network/networkd-ndisc.c
View File

@ -292,14 +292,12 @@ static int ndisc_request_router_route(Route *route, Link *link, sd_ndisc_router
}
static int ndisc_remove_route(Route *route, Link *link) {
int r;
int r, ret = 0;
assert(route);
assert(link);
assert(link->manager);
ndisc_set_route_priority(link, route);
if (!route->table_set)
route->table = link_get_ndisc_route_table(link);
@ -331,9 +329,7 @@ static int ndisc_remove_route(Route *route, Link *link) {
if (existing->source == NETWORK_CONFIG_SOURCE_STATIC)
continue;
r = route_remove_and_cancel(existing, link->manager);
if (r < 0)
return r;
RET_GATHER(ret, route_remove_and_cancel(existing, link->manager));
}
/* Then, check if the route exists. */
@ -341,13 +337,11 @@ static int ndisc_remove_route(Route *route, Link *link) {
if (existing->source == NETWORK_CONFIG_SOURCE_STATIC)
continue;
r = route_remove_and_cancel(existing, link->manager);
if (r < 0)
return r;
RET_GATHER(ret, route_remove_and_cancel(existing, link->manager));
}
}
return 0;
return ret;
}
static int ndisc_address_handler(sd_netlink *rtnl, sd_netlink_message *m, Request *req, Link *link, Address *address) {