ubus: Correct usage of timeout on poll function
[project/ubus.git] / ubusd.c
diff --git a/ubusd.c b/ubusd.c
index 59dee3e..6629720 100644 (file)
--- a/ubusd.c
+++ b/ubusd.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2011 Felix Fietkau <nbd@openwrt.org>
+ * Copyright (C) 2011-2014 Felix Fietkau <nbd@openwrt.org>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU Lesser General Public License version 2.1
 #include <sys/socket.h>
 #include <sys/stat.h>
 #include <sys/uio.h>
+#ifdef FreeBSD
+#include <sys/param.h>
+#endif
+#include <syslog.h>
 #include <signal.h>
 #include <stdio.h>
 #include <unistd.h>
 
 #include "ubusd.h"
 
-static struct ubus_msg_buf *ubus_msg_unshare(struct ubus_msg_buf *ub)
-{
-       ub = realloc(ub, sizeof(*ub) + ub->len);
-       if (!ub)
-               return NULL;
-
-       ub->refcount = 1;
-       memcpy(ub + 1, ub->data, ub->len);
-       ub->data = (void *) (ub + 1);
-       return ub;
-}
-
 static struct ubus_msg_buf *ubus_msg_ref(struct ubus_msg_buf *ub)
 {
        if (ub->refcount == ~0)
-               return ubus_msg_unshare(ub);
+               return ubus_msg_new(ub->data, ub->len, false);
 
        ub->refcount++;
        return ub;
@@ -59,6 +51,8 @@ struct ubus_msg_buf *ubus_msg_new(void *data, int len, bool shared)
        if (!ub)
                return NULL;
 
+       ub->fd = -1;
+
        if (shared) {
                ub->refcount = ~0;
                ub->data = data;
@@ -78,6 +72,9 @@ void ubus_msg_free(struct ubus_msg_buf *ub)
        switch (ub->refcount) {
        case 1:
        case ~0:
+               if (ub->fd >= 0)
+                       close(ub->fd);
+
                free(ub);
                break;
        default:
@@ -88,14 +85,37 @@ void ubus_msg_free(struct ubus_msg_buf *ub)
 
 static int ubus_msg_writev(int fd, struct ubus_msg_buf *ub, int offset)
 {
-       struct iovec iov[2];
+       static struct iovec iov[2];
+       static struct {
+               struct cmsghdr h;
+               int fd;
+       } fd_buf = {
+               .h = {
+                       .cmsg_len = sizeof(fd_buf),
+                       .cmsg_level = SOL_SOCKET,
+                       .cmsg_type = SCM_RIGHTS,
+               },
+       };
+       struct msghdr msghdr = {
+               .msg_iov = iov,
+               .msg_iovlen = ARRAY_SIZE(iov),
+               .msg_control = &fd_buf,
+               .msg_controllen = sizeof(fd_buf),
+       };
+
+       fd_buf.fd = ub->fd;
+       if (ub->fd < 0) {
+               msghdr.msg_control = NULL;
+               msghdr.msg_controllen = 0;
+       }
 
        if (offset < sizeof(ub->hdr)) {
                iov[0].iov_base = ((char *) &ub->hdr) + offset;
                iov[0].iov_len = sizeof(ub->hdr) - offset;
                iov[1].iov_base = (char *) ub->data;
                iov[1].iov_len = ub->len;
-               return writev(fd, iov, 2);
+
+               return sendmsg(fd, &msghdr, 0);
        } else {
                offset -= sizeof(ub->hdr);
                return write(fd, ((char *) ub->data) + offset, ub->len - offset);
@@ -116,6 +136,9 @@ void ubus_msg_send(struct ubus_client *cl, struct ubus_msg_buf *ub, bool free)
 {
        int written;
 
+       if (ub->hdr.type != UBUS_MSG_MONITOR)
+               ubusd_monitor_message(cl, ub, true);
+
        if (!cl->tx_queue[cl->txq_cur]) {
                written = ubus_msg_writev(cl->sock.fd, ub, 0);
                if (written >= ub->len + sizeof(ub->hdr))
@@ -159,7 +182,10 @@ static void handle_client_disconnect(struct ubus_client *cl)
        while (ubus_msg_head(cl))
                ubus_msg_dequeue(cl);
 
+       ubusd_monitor_disconnect(cl);
        ubusd_proto_free_client(cl);
+       if (cl->pending_msg_fd >= 0)
+               close(cl->pending_msg_fd);
        uloop_fd_delete(&cl->sock);
        close(cl->sock.fd);
        free(cl);
@@ -169,6 +195,21 @@ static void client_cb(struct uloop_fd *sock, unsigned int events)
 {
        struct ubus_client *cl = container_of(sock, struct ubus_client, sock);
        struct ubus_msg_buf *ub;
+       static struct iovec iov;
+       static struct {
+               struct cmsghdr h;
+               int fd;
+       } fd_buf = {
+               .h = {
+                       .cmsg_type = SCM_RIGHTS,
+                       .cmsg_level = SOL_SOCKET,
+                       .cmsg_len = sizeof(fd_buf),
+               }
+       };
+       struct msghdr msghdr = {
+               .msg_iov = &iov,
+               .msg_iovlen = 1,
+       };
 
        /* first try to tx more pending data */
        while ((ub = ubus_msg_head(cl))) {
@@ -203,10 +244,26 @@ retry:
                int offset = cl->pending_msg_offset;
                int bytes;
 
-               bytes = read(sock->fd, (char *)&cl->hdrbuf + offset, sizeof(cl->hdrbuf) - offset);
+               fd_buf.fd = -1;
+
+               iov.iov_base = ((char *) &cl->hdrbuf) + offset;
+               iov.iov_len = sizeof(cl->hdrbuf) - offset;
+
+               if (cl->pending_msg_fd < 0) {
+                       msghdr.msg_control = &fd_buf;
+                       msghdr.msg_controllen = sizeof(fd_buf);
+               } else {
+                       msghdr.msg_control = NULL;
+                       msghdr.msg_controllen = 0;
+               }
+
+               bytes = recvmsg(sock->fd, &msghdr, 0);
                if (bytes < 0)
                        goto out;
 
+               if (fd_buf.fd >= 0)
+                       cl->pending_msg_fd = fd_buf.fd;
+
                cl->pending_msg_offset += bytes;
                if (cl->pending_msg_offset < sizeof(cl->hdrbuf))
                        goto out;
@@ -240,8 +297,11 @@ retry:
                }
 
                /* accept message */
+               ub->fd = cl->pending_msg_fd;
+               cl->pending_msg_fd = -1;
                cl->pending_msg_offset = 0;
                cl->pending_msg = NULL;
+               ubusd_monitor_message(cl, ub, false);
                ubusd_proto_receive_message(cl, ub);
                goto retry;
        }
@@ -301,6 +361,11 @@ static int usage(const char *progname)
        return 1;
 }
 
+static void sighup_handler(int sig)
+{
+       ubusd_acl_load();
+}
+
 int main(int argc, char **argv)
 {
        const char *ubus_socket = UBUS_UNIX_SOCKET;
@@ -308,7 +373,9 @@ int main(int argc, char **argv)
        int ch;
 
        signal(SIGPIPE, SIG_IGN);
+       signal(SIGHUP, sighup_handler);
 
+       openlog("ubusd", LOG_PID, LOG_DAEMON);
        uloop_init();
 
        while ((ch = getopt(argc, argv, "s:")) != -1) {
@@ -322,7 +389,7 @@ int main(int argc, char **argv)
        }
 
        unlink(ubus_socket);
-       umask(0177);
+       umask(0111);
        server_fd.fd = usock(USOCK_UNIX | USOCK_SERVER | USOCK_NONBLOCK, ubus_socket, NULL);
        if (server_fd.fd < 0) {
                perror("usock");
@@ -330,6 +397,7 @@ int main(int argc, char **argv)
                goto out;
        }
        uloop_fd_add(&server_fd, ULOOP_READ | ULOOP_EDGE_TRIGGER);
+       ubusd_acl_load();
 
        uloop_run();
        unlink(ubus_socket);