From f5b1c3f98bf082d4ecf57e8f2e3c61fc993d80e7 Mon Sep 17 00:00:00 2001 From: Simon Chopin Date: Fri, 28 Feb 2020 11:28:07 +0100 Subject: net: new function get_ifindex The index of a given interface is an operation that highly depends on the network namespace we're in. This patch simply cuts out a function to get the index for a given interface name from the session initialization code, and expose it in the ap_net structure. This function can then be used to refresh the index when moving interfaces around. Signed-off-by: Simon Chopin --- accel-pppd/include/ap_net.h | 1 + accel-pppd/net.c | 18 ++++++++++++++++++ accel-pppd/session.c | 14 ++------------ 3 files changed, 21 insertions(+), 12 deletions(-) (limited to 'accel-pppd') diff --git a/accel-pppd/include/ap_net.h b/accel-pppd/include/ap_net.h index 91ebdd5..984c6c1 100644 --- a/accel-pppd/include/ap_net.h +++ b/accel-pppd/include/ap_net.h @@ -31,6 +31,7 @@ struct ap_net { void (*rtnl_put)(struct rtnl_handle *); int (*rtnl_open)(struct rtnl_handle *h, int proto); int (*move_link)(struct ap_net *net, int ifindex); + int (*get_ifindex)(const char * ifname); void (*release)(struct ap_net *net); }; diff --git a/accel-pppd/net.c b/accel-pppd/net.c index e5cf281..9a025ed 100644 --- a/accel-pppd/net.c +++ b/accel-pppd/net.c @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -215,6 +216,22 @@ static int def_move_link(struct ap_net *new_net, int ifindex) #endif } +static int def_get_ifindex(const char *ifname) +{ + struct kern_net *n = container_of(net, typeof(*n), net); + struct ifreq ifr; + + memset(&ifr, 0, sizeof(ifr)); + strcpy(ifr.ifr_name, ifname); + + if (ioctl(n->sock, SIOCGIFINDEX, &ifr)) { + log_ppp_error("ioctl(SIOCGIFINDEX): %s\n", strerror(errno)); + return -1; + } + return ifr.ifr_ifindex; +} + + static void def_release(struct ap_net *d) { struct kern_net *n = container_of(d, typeof(*n), net); @@ -299,6 +316,7 @@ static struct ap_net *alloc_net(const char *name) net->rtnl_put = def_rtnl_put; net->rtnl_open = def_rtnl_open; net->move_link = def_move_link; + net->get_ifindex = def_get_ifindex; net->release = def_release; n->sock = socket(AF_INET, SOCK_DGRAM, 0); diff --git a/accel-pppd/session.c b/accel-pppd/session.c index 8ef569e..1411f24 100644 --- a/accel-pppd/session.c +++ b/accel-pppd/session.c @@ -88,21 +88,11 @@ void __export ap_session_set_ifindex(struct ap_session *ses) int __export ap_session_starting(struct ap_session *ses) { - struct ifreq ifr; - if (ap_shutdown) return -1; - if (ses->ifindex == -1 && ses->ifname[0]) { - memset(&ifr, 0, sizeof(ifr)); - strcpy(ifr.ifr_name, ses->ifname); - - if (net->sock_ioctl(SIOCGIFINDEX, &ifr)) { - log_ppp_error("ioctl(SIOCGIFINDEX): %s\n", strerror(errno)); - return -1; - } - ses->ifindex = ifr.ifr_ifindex; - } + if (ses->ifindex == -1 && ses->ifname[0]) + ses->ifindex = net->get_ifindex(ses->ifname); if (ses->ifindex != -1) ap_session_set_ifindex(ses); -- cgit v1.2.3