X-Git-Url: http://git.archive.openwrt.org/?a=blobdiff_plain;f=system-linux.c;h=9d6451fdbd524a19bf72de93f2cfdad4718bc419;hb=37d529a958dfbcbb35404b0255506690b4537552;hp=44332a2b88d79af2b487b12a966673f9e9bff547;hpb=fb0e4138070d7c2ce723af5780e763af3a1353d8;p=project%2Fnetifd.git

diff --git a/system-linux.c b/system-linux.c
index 44332a2..9d6451f 100644
--- a/system-linux.c
+++ b/system-linux.c
@@ -24,17 +24,29 @@
 #include "device.h"
 #include "system.h"
 
+struct event_socket {
+	struct uloop_fd uloop;
+	struct nl_sock *sock;
+	struct nl_cb *cb;
+};
+
 static int sock_ioctl = -1;
 static struct nl_sock *sock_rtnl = NULL;
-static struct nl_sock *sock_rtnl_event = NULL;
 
-static void handler_rtnl_event(struct uloop_fd *u, unsigned int events);
 static int cb_rtnl_event(struct nl_msg *msg, void *arg);
-static struct uloop_fd rtnl_event = {.cb = handler_rtnl_event};
-static struct nl_cb *nl_cb_rtnl_event;
+
+static void handler_nl_event(struct uloop_fd *u, unsigned int events)
+{
+	struct event_socket *ev = container_of(u, struct event_socket, uloop);
+	nl_recvmsgs(ev->sock, ev->cb);
+}
 
 int system_init(void)
 {
+	static struct event_socket rtnl_event = {
+		.uloop.cb = handler_nl_event,
+	};
+
 	sock_ioctl = socket(AF_LOCAL, SOCK_DGRAM, 0);
 	fcntl(sock_ioctl, F_SETFD, fcntl(sock_ioctl, F_GETFD) | FD_CLOEXEC);
 
@@ -44,47 +56,30 @@ int system_init(void)
 		return -1;
 
 	if (nl_connect(sock_rtnl, NETLINK_ROUTE))
-		goto error_free_sock;
+		return -1;
 
 	// Prepare socket for link events
-	nl_cb_rtnl_event = nl_cb_alloc(NL_CB_DEFAULT);
-	if (!nl_cb_rtnl_event)
-		goto error_free_sock;
+	rtnl_event.cb = nl_cb_alloc(NL_CB_DEFAULT);
+	if (!rtnl_event.cb)
+		return -1;
 
-	nl_cb_set(nl_cb_rtnl_event, NL_CB_VALID, NL_CB_CUSTOM,
+	nl_cb_set(rtnl_event.cb, NL_CB_VALID, NL_CB_CUSTOM,
 		  cb_rtnl_event, NULL);
 
-	sock_rtnl_event = nl_socket_alloc();
-	if (!sock_rtnl_event)
-		goto error_free_cb;
+	rtnl_event.sock = nl_socket_alloc();
+	if (!rtnl_event.sock)
+		return -1;
 
-	if (nl_connect(sock_rtnl_event, NETLINK_ROUTE))
-		goto error_free_event;
+	if (nl_connect(rtnl_event.sock, NETLINK_ROUTE))
+		return -1;
 
 	// Receive network link events form kernel
-	nl_socket_add_membership(sock_rtnl_event, RTNLGRP_LINK);
+	nl_socket_add_membership(rtnl_event.sock, RTNLGRP_LINK);
 
-	rtnl_event.fd = nl_socket_get_fd(sock_rtnl_event);
-	uloop_fd_add(&rtnl_event, ULOOP_READ | ULOOP_EDGE_TRIGGER);
+	rtnl_event.uloop.fd = nl_socket_get_fd(rtnl_event.sock);
+	uloop_fd_add(&rtnl_event.uloop, ULOOP_READ | ULOOP_EDGE_TRIGGER);
 
 	return 0;
-
-error_free_event:
-	nl_socket_free(sock_rtnl_event);
-	sock_rtnl_event = NULL;
-error_free_cb:
-	nl_cb_put(nl_cb_rtnl_event);
-	nl_cb_rtnl_event = NULL;
-error_free_sock:
-	nl_socket_free(sock_rtnl);
-	sock_rtnl = NULL;
-	return -1;
-}
-
-// If socket is ready for reading parse netlink events
-static void handler_rtnl_event(struct uloop_fd *u, unsigned int events)
-{
-	nl_recvmsgs(sock_rtnl_event, nl_cb_rtnl_event);
 }
 
 static void system_set_sysctl(const char *path, const char *val)
@@ -101,7 +96,7 @@ static void system_set_sysctl(const char *path, const char *val)
 
 static void system_set_dev_sysctl(const char *path, const char *device, const char *val)
 {
-	char buf[256];
+	static char buf[256];
 
 	snprintf(buf, sizeof(buf), path, val);
 	system_set_sysctl(buf, val);
@@ -367,7 +362,7 @@ out:
  */
 void system_if_clear_state(struct device *dev)
 {
-	char buf[256];
+	static char buf[256];
 	char *bridge;
 
 	if (dev->external)