|
Zbigniew Jędrzejewski-Szmek |
126222 |
From 0e075f4d98a653cc0b468fc786fcd869075a75e1 Mon Sep 17 00:00:00 2001
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
From: Kay Sievers <kay@vrfy.org>
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Date: Wed, 18 Dec 2013 16:49:19 +0100
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Subject: [PATCH] libudev: devices received from udev are always initialized
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Conflicts:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
TODO
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
---
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
src/libudev/libudev-device.c | 2 ++
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
src/libudev/libudev-monitor.c | 62 +++++++++++++++++++++----------------------
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
2 files changed, 33 insertions(+), 31 deletions(-)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
diff --git a/src/libudev/libudev-device.c b/src/libudev/libudev-device.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
index 161181a..b6c4e24 100644
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
--- a/src/libudev/libudev-device.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+++ b/src/libudev/libudev-device.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -536,6 +536,8 @@ int udev_device_read_db(struct udev_device *udev_device, const char *dbfile)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_dbg(udev_device->udev, "no db file to read %s: %m\n", dbfile);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return -1;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ /* devices with a database entry are initialized */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_device->is_initialized = true;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
while (fgets(line, sizeof(line), f)) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
diff --git a/src/libudev/libudev-monitor.c b/src/libudev/libudev-monitor.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
index 0212792..ff89e14 100644
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
--- a/src/libudev/libudev-monitor.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+++ b/src/libudev/libudev-monitor.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -324,9 +324,6 @@ _public_ int udev_monitor_enable_receiving(struct udev_monitor *udev_monitor)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
int err = 0;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
const int on = 1;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_monitor->snl.nl.nl_family == 0)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- return -EINVAL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_monitor_filter_update(udev_monitor);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (!udev_monitor->bound) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -524,7 +521,6 @@ _public_ struct udev_device *udev_monitor_receive_device(struct udev_monitor *ud
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
char buf[8192];
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
ssize_t buflen;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
ssize_t bufpos;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- struct udev_monitor_netlink_header *nlh;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
retry:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (udev_monitor == NULL)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -536,11 +532,8 @@ retry:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
smsg.msg_iovlen = 1;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
smsg.msg_control = cred_msg;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
smsg.msg_controllen = sizeof(cred_msg);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_monitor->snl.nl.nl_family != 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- smsg.msg_name = &sn;;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- smsg.msg_namelen = sizeof(snl);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- }
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ smsg.msg_name = &sn;;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ smsg.msg_namelen = sizeof(snl);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
buflen = recvmsg(udev_monitor->sock, &smsg, 0);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (buflen < 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -554,20 +547,18 @@ retry:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_monitor->snl.nl.nl_family != 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (snl.nl.nl_groups == 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- /* unicast message, check if we trust the sender */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_monitor->snl_trusted_sender.nl.nl_pid == 0 ||
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- snl.nl.nl_pid != udev_monitor->snl_trusted_sender.nl.nl_pid) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- udev_dbg(udev_monitor->udev, "unicast netlink message ignored\n");
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- }
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- } else if (snl.nl.nl_groups == UDEV_MONITOR_KERNEL) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (snl.nl.nl_pid > 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- udev_dbg(udev_monitor->udev, "multicast kernel netlink message from pid %d ignored\n",
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- snl.nl.nl_pid);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- }
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (snl.nl.nl_groups == 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ /* unicast message, check if we trust the sender */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (udev_monitor->snl_trusted_sender.nl.nl_pid == 0 ||
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ snl.nl.nl_pid != udev_monitor->snl_trusted_sender.nl.nl_pid) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_dbg(udev_monitor->udev, "unicast netlink message ignored\n");
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ }
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ } else if (snl.nl.nl_groups == UDEV_MONITOR_KERNEL) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (snl.nl.nl_pid > 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_dbg(udev_monitor->udev, "multicast kernel netlink message from pid %d ignored\n",
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ snl.nl.nl_pid);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -583,35 +574,47 @@ retry:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device = udev_device_new(udev_monitor->udev);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (udev_device == NULL)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (memcmp(buf, "libudev", 8) == 0) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ struct udev_monitor_netlink_header *nlh;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
/* udev message needs proper version magic */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
nlh = (struct udev_monitor_netlink_header *) buf;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (nlh->magic != htonl(UDEV_MONITOR_MAGIC)) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_err(udev_monitor->udev, "unrecognized message signature (%x != %x)\n",
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- nlh->magic, htonl(UDEV_MONITOR_MAGIC));
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ nlh->magic, htonl(UDEV_MONITOR_MAGIC));
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device_unref(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (nlh->properties_off+32 > (size_t)buflen)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (nlh->properties_off+32 > (size_t)buflen) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device_unref(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ }
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
bufpos = nlh->properties_off;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ /* devices received from udev are always initialized */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device_set_is_initialized(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
} else {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
/* kernel message with header */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
bufpos = strlen(buf) + 1;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if ((size_t)bufpos < sizeof("a@/d") || bufpos >= buflen) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_dbg(udev_monitor->udev, "invalid message length\n");
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device_unref(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
/* check message header */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (strstr(buf, "@/") == NULL) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_dbg(udev_monitor->udev, "unrecognized message header\n");
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ udev_device_unref(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
}
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- udev_device = udev_device_new(udev_monitor->udev);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_device == NULL)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
udev_device_set_info_loaded(udev_device);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
while (bufpos < buflen) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -664,9 +667,6 @@ int udev_monitor_send_device(struct udev_monitor *udev_monitor,
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
struct udev_list_entry *list_entry;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
uint64_t tag_bloom_bits;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- if (udev_monitor->snl.nl.nl_family == 0)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
- return -EINVAL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
-
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
blen = udev_device_get_properties_monitor_buf(udev_device, &buf;;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (blen < 32)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return -EINVAL;
|