From d22f1006a1910e32ff099330d70badaf2aec75c6 Mon Sep 17 00:00:00 2001 From: Tom Gundersen Date: Thu, 6 Mar 2014 19:44:37 +0100 Subject: [PATCH 1/1] sd-rtnl/networkd: use new rtnl_message_read() API and drop helper With the new sd_rtnl_message_read_string(), there is no longer a need for rtnl_message_get_ifname(). --- src/libsystemd/sd-rtnl/rtnl-util.c | 16 ---------------- src/libsystemd/sd-rtnl/rtnl-util.h | 2 -- src/network/networkd-manager.c | 4 ++-- 3 files changed, 2 insertions(+), 20 deletions(-) diff --git a/src/libsystemd/sd-rtnl/rtnl-util.c b/src/libsystemd/sd-rtnl/rtnl-util.c index fc834e955..c8b20d109 100644 --- a/src/libsystemd/sd-rtnl/rtnl-util.c +++ b/src/libsystemd/sd-rtnl/rtnl-util.c @@ -153,19 +153,3 @@ bool rtnl_message_type_is_addr(uint16_t type) { return false; } } - -int rtnl_message_link_get_ifname(sd_rtnl_message *message, const char **ret) { - unsigned short type; - void *name; - - assert(rtnl_message_type_is_link(message->hdr->nlmsg_type)); - - while (sd_rtnl_message_read(message, &type, &name)) { - if (type == IFLA_IFNAME) { - *ret = name; - return 0; - } - } - - return -ENOENT; -} diff --git a/src/libsystemd/sd-rtnl/rtnl-util.h b/src/libsystemd/sd-rtnl/rtnl-util.h index 33746afe2..7fe922272 100644 --- a/src/libsystemd/sd-rtnl/rtnl-util.h +++ b/src/libsystemd/sd-rtnl/rtnl-util.h @@ -37,8 +37,6 @@ bool rtnl_message_type_is_route(uint16_t type); int rtnl_set_link_name(sd_rtnl *rtnl, int ifindex, const char *name); int rtnl_set_link_properties(sd_rtnl *rtnl, int ifindex, const char *alias, const struct ether_addr *mac, unsigned mtu); -int rtnl_message_link_get_ifname(sd_rtnl_message *m, const char **ret); - DEFINE_TRIVIAL_CLEANUP_FUNC(sd_rtnl*, sd_rtnl_unref); DEFINE_TRIVIAL_CLEANUP_FUNC(sd_rtnl_message*, sd_rtnl_message_unref); diff --git a/src/network/networkd-manager.c b/src/network/networkd-manager.c index c89adfba4..f41914f88 100644 --- a/src/network/networkd-manager.c +++ b/src/network/networkd-manager.c @@ -303,7 +303,7 @@ int manager_udev_listen(Manager *m) { static int manager_rtnl_process_link(sd_rtnl *rtnl, sd_rtnl_message *message, void *userdata) { Manager *m = userdata; Link *link; - const char *name; + char *name; int r, ifindex; assert(rtnl); @@ -316,7 +316,7 @@ static int manager_rtnl_process_link(sd_rtnl *rtnl, sd_rtnl_message *message, vo return 0; } - r = rtnl_message_link_get_ifname(message, &name); + r = sd_rtnl_message_read_string(message, IFLA_IFNAME, &name); if (r < 0) log_debug("received RTM_NEWLINK message without valid IFLA_IFNAME"); else { -- 2.30.2