add .gitignore

[project/netifd.git] / system-linux.c

diff --git a/system-linux.c b/system-linux.c
index 488f5be..158045a 100644 (file)
--- a/system-linux.c
+++ b/system-linux.c
@@ -10,6 +10,7 @@
 
 #include <netlink/msg.h>
 #include <netlink/attr.h>
+#include <netlink/socket.h>
 #include <libubox/uloop.h>
 
 #include "netifd.h"
@@ -31,42 +32,46 @@ int system_init(void)
        fcntl(sock_ioctl, F_SETFD, fcntl(sock_ioctl, F_GETFD) | FD_CLOEXEC);
 
        // Prepare socket for routing / address control
-       if ((sock_rtnl = nl_socket_alloc())) {
-               if (nl_connect(sock_rtnl, NETLINK_ROUTE)) {
-                       nl_socket_free(sock_rtnl);
-                       sock_rtnl = NULL;
-               }
-       }
+       sock_rtnl = nl_socket_alloc();
+       if (!sock_rtnl)
+               return -1;
+
+       if (nl_connect(sock_rtnl, NETLINK_ROUTE))
+               goto error_free_sock;
 
        // Prepare socket for link events
-       if ((nl_cb_rtnl_event = nl_cb_alloc(NL_CB_DEFAULT)))
-               nl_cb_set(nl_cb_rtnl_event, NL_CB_VALID, NL_CB_CUSTOM,
-                                                       cb_rtnl_event, NULL);
-
-       if (nl_cb_rtnl_event && (sock_rtnl_event = nl_socket_alloc())) {
-               if (nl_connect(sock_rtnl_event, NETLINK_ROUTE)) {
-                       nl_socket_free(sock_rtnl_event);
-                       sock_rtnl_event = NULL;
-               }
-               // Receive network link events form kernel
-               nl_socket_add_membership(sock_rtnl_event, RTNLGRP_LINK);
-
-               // Synthesize initial link messages
-               struct nl_msg *m = nlmsg_alloc_simple(RTM_GETLINK, NLM_F_DUMP);
-               if (m && nlmsg_reserve(m, sizeof(struct ifinfomsg), 0)) {
-                       nl_send_auto_complete(sock_rtnl_event, m);
-                       nlmsg_free(m);
-               }
-
-#ifdef NLA_PUT_DATA
-               rtnl_event.fd = nl_socket_get_fd(sock_rtnl_event);
-#else
-               rtnl_event.fd = sock_rtnl_event->s_fd; // libnl-tiny hack...
-#endif
-               uloop_fd_add(&rtnl_event, ULOOP_READ | ULOOP_EDGE_TRIGGER);
-       }
+       nl_cb_rtnl_event = nl_cb_alloc(NL_CB_DEFAULT);
+       if (!nl_cb_rtnl_event)
+               goto error_free_sock;
+
+       nl_cb_set(nl_cb_rtnl_event, NL_CB_VALID, NL_CB_CUSTOM,
+                 cb_rtnl_event, NULL);
+
+       sock_rtnl_event = nl_socket_alloc();
+       if (!sock_rtnl_event)
+               goto error_free_cb;
+
+       if (nl_connect(sock_rtnl_event, NETLINK_ROUTE))
+               goto error_free_event;
+
+       // Receive network link events form kernel
+       nl_socket_add_membership(sock_rtnl_event, RTNLGRP_LINK);
+
+       rtnl_event.fd = nl_socket_get_fd(sock_rtnl_event);
+       uloop_fd_add(&rtnl_event, ULOOP_READ | ULOOP_EDGE_TRIGGER);
 
        return -(sock_ioctl < 0 || !sock_rtnl);
+
+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
@@ -189,7 +194,8 @@ int system_if_down(struct device *dev)
 
 int system_if_check(struct device *dev)
 {
-       return -!(system_if_resolve(dev));
+       device_set_present(dev, (system_if_resolve(dev) >= 0));
+       return 0;
 }
 
 static int system_addr(struct device *dev, struct device_addr *addr, int cmd)