chiark / gitweb /
libudev: make list_node functions available
[elogind.git] / udev / lib / libudev-ctrl.c
index f7cec371b1037377d3ffeefbc5d9b7a5a7e94749..848f5078783db4a2a4080df73b7eb08083c9d3b0 100644 (file)
@@ -17,8 +17,6 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "config.h"
-
 #include <errno.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -32,7 +30,8 @@
 #include "libudev.h"
 #include "libudev-private.h"
 
-#define UDEV_CTRL_MAGIC                                "udevd-128"
+/* last known version with this wire protocol */
+#define UDEV_CTRL_MAGIC                                0xdead1dea
 
 enum udev_ctrl_msg_type {
        UDEV_CTRL_UNKNOWN,
@@ -45,8 +44,9 @@ enum udev_ctrl_msg_type {
        UDEV_CTRL_SET_MAX_CHILDS_RUNNING,
 };
 
-struct ctrl_msg_wire {
-       char magic[32];
+struct udev_ctrl_msg_wire {
+       char version[16];
+       unsigned int magic;
        enum udev_ctrl_msg_type type;
        union {
                int intval;
@@ -57,7 +57,7 @@ struct ctrl_msg_wire {
 struct udev_ctrl_msg {
        int refcount;
        struct udev_ctrl *uctrl;
-       struct ctrl_msg_wire ctrl_msg_wire;
+       struct udev_ctrl_msg_wire ctrl_msg_wire;
 };
 
 struct udev_ctrl {
@@ -81,7 +81,7 @@ struct udev_ctrl *udev_ctrl_new_from_socket(struct udev *udev, const char *socke
 
        uctrl->sock = socket(AF_LOCAL, SOCK_DGRAM, 0);
        if (uctrl->sock < 0) {
-               err(udev, "error getting socket: %s\n", strerror(errno));
+               err(udev, "error getting socket: %m\n");
                udev_ctrl_unref(uctrl);
                return NULL;
        }
@@ -103,7 +103,7 @@ int udev_ctrl_enable_receiving(struct udev_ctrl *uctrl)
 
        err= bind(uctrl->sock, (struct sockaddr *)&uctrl->saddr, uctrl->addrlen);
        if (err < 0) {
-               err(uctrl->udev, "bind failed: %s\n", strerror(errno));
+               err(uctrl->udev, "bind failed: %m\n");
                return err;
        }
 
@@ -146,11 +146,12 @@ int udev_ctrl_get_fd(struct udev_ctrl *uctrl)
 
 static int ctrl_send(struct udev_ctrl *uctrl, enum udev_ctrl_msg_type type, int intval, const char *buf)
 {
-       struct ctrl_msg_wire ctrl_msg_wire;
+       struct udev_ctrl_msg_wire ctrl_msg_wire;
        int err;
 
-       memset(&ctrl_msg_wire, 0x00, sizeof(struct ctrl_msg_wire));
-       strcpy(ctrl_msg_wire.magic, UDEV_CTRL_MAGIC);
+       memset(&ctrl_msg_wire, 0x00, sizeof(struct udev_ctrl_msg_wire));
+       strcpy(ctrl_msg_wire.version, "udev-" VERSION);
+       ctrl_msg_wire.magic = UDEV_CTRL_MAGIC;
        ctrl_msg_wire.type = type;
 
        if (buf != NULL)
@@ -158,9 +159,10 @@ static int ctrl_send(struct udev_ctrl *uctrl, enum udev_ctrl_msg_type type, int
        else
                ctrl_msg_wire.intval = intval;
 
-       err = sendto(uctrl->sock, &ctrl_msg_wire, sizeof(ctrl_msg_wire), 0, (struct sockaddr *)&uctrl->saddr, uctrl->addrlen);
+       err = sendto(uctrl->sock, &ctrl_msg_wire, sizeof(ctrl_msg_wire), 0,
+                    (struct sockaddr *)&uctrl->saddr, uctrl->addrlen);
        if (err == -1) {
-               err(uctrl->udev, "error sending message: %s\n", strerror(errno));
+               err(uctrl->udev, "error sending message: %m\n");
        }
        return err;
 }
@@ -191,7 +193,7 @@ int udev_ctrl_send_reload_rules(struct udev_ctrl *uctrl)
 
 int udev_ctrl_send_set_env(struct udev_ctrl *uctrl, const char *key)
 {
-       ctrl_send(uctrl, UDEV_CTRL_SET_ENV, 0, optarg);
+       ctrl_send(uctrl, UDEV_CTRL_SET_ENV, 0, key);
        return 0;
 }
 
@@ -219,7 +221,7 @@ struct udev_ctrl_msg *udev_ctrl_receive_msg(struct udev_ctrl *uctrl)
        uctrl_msg->uctrl = uctrl;
 
        iov.iov_base = &uctrl_msg->ctrl_msg_wire;
-       iov.iov_len = sizeof(struct udev_ctrl_msg);
+       iov.iov_len = sizeof(struct udev_ctrl_msg_wire);
 
        memset(&smsg, 0x00, sizeof(struct msghdr));
        smsg.msg_iov = &iov;
@@ -229,7 +231,7 @@ struct udev_ctrl_msg *udev_ctrl_receive_msg(struct udev_ctrl *uctrl)
 
        size = recvmsg(uctrl->sock, &smsg, 0);
        if (size <  0) {
-               err(uctrl->udev, "unable to receive user udevd message: %s\n", strerror(errno));
+               err(uctrl->udev, "unable to receive user udevd message: %m\n");
                goto err;
        }
        cmsg = CMSG_FIRSTHDR(&smsg);
@@ -245,8 +247,8 @@ struct udev_ctrl_msg *udev_ctrl_receive_msg(struct udev_ctrl *uctrl)
                goto err;
        }
 
-       if (strncmp(uctrl_msg->ctrl_msg_wire.magic, UDEV_CTRL_MAGIC, sizeof(UDEV_CTRL_MAGIC)) != 0 ) {
-               err(uctrl->udev, "message magic '%s' doesn't match, ignore it\n", uctrl_msg->ctrl_msg_wire.magic);
+       if (uctrl_msg->ctrl_msg_wire.magic != UDEV_CTRL_MAGIC) {
+               err(uctrl->udev, "message magic 0x%08x doesn't match, ignore it\n", uctrl_msg->ctrl_msg_wire.magic);
                goto err;
        }