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 <s.chopin@alphalink.fr>

3 files changed, 21 insertions, 12 deletions

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 <sched.h>
 #include <limits.h>
 #include <errno.h>
+#include <linux/if.h>
 #include <sys/socket.h>
 #include <sys/ioctl.h>
 #include <sys/syscall.h>
@@ -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);