ryantimwilson / rpms / systemd

Forked from rpms/systemd 6 days ago
Clone
Zbigniew Jędrzejewski-Szmek 62fe94
From 4d6dac13ad376cb537647741c45185395beb3e9c Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Mon, 15 Sep 2014 12:04:29 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] udev: apply permissions to static nodes before signallying
Zbigniew Jędrzejewski-Szmek 62fe94
 READY
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Processes expecting static nodes to have the right permissions may order themselves after systemd-udevd.service,
Zbigniew Jędrzejewski-Szmek 62fe94
make sure that actually guarantees what is expected.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/udev/udevd.c | 28 ++++++++++++++--------------
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 14 insertions(+), 14 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
index cfa071eba9..e8b3602986 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/udev/udevd.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1199,6 +1199,20 @@ int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         udev_monitor_set_receive_buffer_size(monitor, 128 * 1024 * 1024);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
+        log_info("starting version " VERSION "\n");
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        udev_builtin_init(udev);
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        rules = udev_rules_new(udev, arg_resolve_names);
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (rules == NULL) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_error("error reading rules");
Zbigniew Jędrzejewski-Szmek 62fe94
+                goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        rc = udev_rules_apply_static_dev_perms(rules);
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (rc < 0)
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_error("failed to apply permissions on static device nodes - %s", strerror(-rc));
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
         if (arg_daemonize) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 pid_t pid;
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1222,20 +1236,6 @@ int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 sd_notify(1, "READY=1");
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        log_info("starting version " VERSION "\n");
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
-        udev_builtin_init(udev);
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
-        rules = udev_rules_new(udev, arg_resolve_names);
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (rules == NULL) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                log_error("error reading rules");
Zbigniew Jędrzejewski-Szmek 62fe94
-                goto exit;
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
-        rc = udev_rules_apply_static_dev_perms(rules);
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (rc < 0)
Zbigniew Jędrzejewski-Szmek 62fe94
-                log_error("failed to apply permissions on static device nodes - %s", strerror(-rc));
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
         if (arg_children_max <= 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 cpu_set_t cpu_set;
Zbigniew Jędrzejewski-Szmek 62fe94