diff options
author | Yu Watanabe <watanabe.yu+github@gmail.com> | 2020-09-29 09:20:00 +0200 |
---|---|---|
committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2020-10-06 19:36:29 +0200 |
commit | 8dfed23d92ebf3144981dcb28765a24b4417e877 (patch) | |
tree | 4d9a175a49fab10667c5c8a14d845b653f43a694 /src | |
parent | sd-netlink: fix typo and make netlink_add_match() take description (diff) | |
download | systemd-8dfed23d92ebf3144981dcb28765a24b4417e877.tar.xz systemd-8dfed23d92ebf3144981dcb28765a24b4417e877.zip |
network: use typesafe macro netlink_add_match()
Diffstat (limited to 'src')
-rw-r--r-- | src/network/networkd-manager.c | 43 | ||||
-rw-r--r-- | src/network/networkd-manager.h | 6 |
2 files changed, 19 insertions, 30 deletions
diff --git a/src/network/networkd-manager.c b/src/network/networkd-manager.c index ad1c97b83e..f24da90f33 100644 --- a/src/network/networkd-manager.c +++ b/src/network/networkd-manager.c @@ -283,10 +283,9 @@ static int manager_connect_udev(Manager *m) { return 0; } -int manager_rtnl_process_route(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { +static int manager_rtnl_process_route(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { _cleanup_(route_freep) Route *tmp = NULL; Route *route = NULL; - Manager *m = userdata; Link *link = NULL; uint32_t ifindex; uint16_t type; @@ -579,8 +578,7 @@ static int manager_rtnl_process_neighbor_lladdr(sd_netlink_message *message, uni return r; } -int manager_rtnl_process_neighbor(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { - Manager *m = userdata; +static int manager_rtnl_process_neighbor(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { Link *link = NULL; Neighbor *neighbor = NULL; int ifindex, family, r; @@ -719,9 +717,8 @@ int manager_rtnl_process_neighbor(sd_netlink *rtnl, sd_netlink_message *message, return 1; } -int manager_rtnl_process_address(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { +int manager_rtnl_process_address(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { _cleanup_free_ char *buf = NULL; - Manager *m = userdata; Link *link = NULL; uint16_t type; unsigned char flags, prefixlen, scope; @@ -882,8 +879,7 @@ int manager_rtnl_process_address(sd_netlink *rtnl, sd_netlink_message *message, return 1; } -static int manager_rtnl_process_link(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { - Manager *m = userdata; +static int manager_rtnl_process_link(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { Link *link = NULL; NetDev *netdev = NULL; uint16_t type; @@ -970,20 +966,18 @@ static int manager_rtnl_process_link(sd_netlink *rtnl, sd_netlink_message *messa return 1; } -int manager_rtnl_process_rule(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { +static int manager_rtnl_process_rule(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { _cleanup_(routing_policy_rule_freep) RoutingPolicyRule *tmp = NULL; _cleanup_free_ char *from = NULL, *to = NULL; RoutingPolicyRule *rule = NULL; const char *iif = NULL, *oif = NULL; uint32_t suppress_prefixlen; - Manager *m = userdata; unsigned flags; uint16_t type; int r; assert(rtnl); assert(message); - assert(m); if (sd_netlink_message_is_error(message)) { r = sd_netlink_message_get_errno(message); @@ -1202,11 +1196,10 @@ int manager_rtnl_process_rule(sd_netlink *rtnl, sd_netlink_message *message, voi return 1; } -int manager_rtnl_process_nexthop(sd_netlink *rtnl, sd_netlink_message *message, void *userdata) { +static int manager_rtnl_process_nexthop(sd_netlink *rtnl, sd_netlink_message *message, Manager *m) { _cleanup_(nexthop_freep) NextHop *tmp = NULL; _cleanup_free_ char *gateway = NULL; NextHop *nexthop = NULL; - Manager *m = userdata; Link *link = NULL; uint16_t type; int r; @@ -1387,51 +1380,51 @@ static int manager_connect_rtnl(Manager *m) { if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWLINK, &manager_rtnl_process_link, NULL, m, "network-rtnl_process_link"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWLINK, &manager_rtnl_process_link, NULL, m, "network-rtnl_process_link"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELLINK, &manager_rtnl_process_link, NULL, m, "network-rtnl_process_link"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELLINK, &manager_rtnl_process_link, NULL, m, "network-rtnl_process_link"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWADDR, &manager_rtnl_process_address, NULL, m, "network-rtnl_process_address"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWADDR, &manager_rtnl_process_address, NULL, m, "network-rtnl_process_address"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELADDR, &manager_rtnl_process_address, NULL, m, "network-rtnl_process_address"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELADDR, &manager_rtnl_process_address, NULL, m, "network-rtnl_process_address"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWNEIGH, &manager_rtnl_process_neighbor, NULL, m, "network-rtnl_process_neighbor"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWNEIGH, &manager_rtnl_process_neighbor, NULL, m, "network-rtnl_process_neighbor"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELNEIGH, &manager_rtnl_process_neighbor, NULL, m, "network-rtnl_process_neighbor"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELNEIGH, &manager_rtnl_process_neighbor, NULL, m, "network-rtnl_process_neighbor"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWROUTE, &manager_rtnl_process_route, NULL, m, "network-rtnl_process_route"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWROUTE, &manager_rtnl_process_route, NULL, m, "network-rtnl_process_route"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELROUTE, &manager_rtnl_process_route, NULL, m, "network-rtnl_process_route"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELROUTE, &manager_rtnl_process_route, NULL, m, "network-rtnl_process_route"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWRULE, &manager_rtnl_process_rule, NULL, m, "network-rtnl_process_rule"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWRULE, &manager_rtnl_process_rule, NULL, m, "network-rtnl_process_rule"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELRULE, &manager_rtnl_process_rule, NULL, m, "network-rtnl_process_rule"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELRULE, &manager_rtnl_process_rule, NULL, m, "network-rtnl_process_rule"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_NEWNEXTHOP, &manager_rtnl_process_nexthop, NULL, m, "network-rtnl_process_nexthop"); + r = netlink_add_match(m->rtnl, NULL, RTM_NEWNEXTHOP, &manager_rtnl_process_nexthop, NULL, m, "network-rtnl_process_nexthop"); if (r < 0) return r; - r = sd_netlink_add_match(m->rtnl, NULL, RTM_DELNEXTHOP, &manager_rtnl_process_nexthop, NULL, m, "network-rtnl_process_nexthop"); + r = netlink_add_match(m->rtnl, NULL, RTM_DELNEXTHOP, &manager_rtnl_process_nexthop, NULL, m, "network-rtnl_process_nexthop"); if (r < 0) return r; diff --git a/src/network/networkd-manager.h b/src/network/networkd-manager.h index 255a0d965f..1b6b7f785d 100644 --- a/src/network/networkd-manager.h +++ b/src/network/networkd-manager.h @@ -89,11 +89,7 @@ int manager_rtnl_enumerate_routes(Manager *m); int manager_rtnl_enumerate_rules(Manager *m); int manager_rtnl_enumerate_nexthop(Manager *m); -int manager_rtnl_process_address(sd_netlink *nl, sd_netlink_message *message, void *userdata); -int manager_rtnl_process_neighbor(sd_netlink *nl, sd_netlink_message *message, void *userdata); -int manager_rtnl_process_route(sd_netlink *nl, sd_netlink_message *message, void *userdata); -int manager_rtnl_process_rule(sd_netlink *nl, sd_netlink_message *message, void *userdata); -int manager_rtnl_process_nexthop(sd_netlink *nl, sd_netlink_message *message, void *userdata); +int manager_rtnl_process_address(sd_netlink *nl, sd_netlink_message *message, Manager *m); void manager_dirty(Manager *m); |