[kmod/f21] Backport patch to fix device node permissions (rhbz 1147248)

Josh Boyer jwboyer at fedoraproject.org
Wed Oct 29 18:09:58 UTC 2014


commit 722e0ca440d3f10fe0fd99102b8c49b59887a361
Author: Josh Boyer <jwboyer at fedoraproject.org>
Date:   Wed Oct 29 14:07:48 2014 -0400

    Backport patch to fix device node permissions (rhbz 1147248)

 ...s-indicate-that-creation-of-static-nodes-.patch |   36 ++++++++++++++++++++
 kmod.spec                                          |    8 ++++-
 2 files changed, 43 insertions(+), 1 deletions(-)
---
diff --git a/0001-static-nodes-indicate-that-creation-of-static-nodes-.patch b/0001-static-nodes-indicate-that-creation-of-static-nodes-.patch
new file mode 100644
index 0000000..f8cb8f7
--- /dev/null
+++ b/0001-static-nodes-indicate-that-creation-of-static-nodes-.patch
@@ -0,0 +1,36 @@
+From 27eceb2e4ecbf2e31940b67cee682a6935f61e6c Mon Sep 17 00:00:00 2001
+From: Tom Gundersen <teg at jklm.no>
+Date: Mon, 27 Oct 2014 17:55:03 +0100
+Subject: [PATCH] static-nodes: indicate that creation of static nodes should
+ only happen at boot
+
+udev will only manage static nodes that exist at the time udev is started, so
+creating static nodes later on will likely not behave as expected. In
+particular, recreating the static nodes at run-time will reset any permissions
+udev may have applied to the nodes at boot.
+
+See <https://bugzilla.redhat.com/show_bug.cgi?id=1147248> and the discussion
+following <http://permalink.gmane.org/gmane.comp.sysutils.systemd.devel/23795>.
+
+Note that this requires (the yet to be released) systemd v217 or a backport
+of systemd patch 8c94052ee543c3598a3c7b0c46688150aa2c6168.
+---
+ tools/static-nodes.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/tools/static-nodes.c b/tools/static-nodes.c
+index d1e3b6e..8d2356d 100644
+--- a/tools/static-nodes.c
++++ b/tools/static-nodes.c
+@@ -96,7 +96,7 @@ static int write_tmpfiles(FILE *out, char modname[], char devname[], char type,
+ 			return EXIT_FAILURE;
+ 	}
+ 
+-	ret = fprintf(out, "%c /dev/%s 0600 - - - %u:%u\n",
++	ret = fprintf(out, "%c! /dev/%s 0600 - - - %u:%u\n",
+ 		      type, devname, maj, min);
+ 	if (ret < 0)
+ 		return EXIT_FAILURE;
+-- 
+1.9.3
+
diff --git a/kmod.spec b/kmod.spec
index 77efb88..5a5075c 100644
--- a/kmod.spec
+++ b/kmod.spec
@@ -1,6 +1,6 @@
 Name:		kmod
 Version:	18
-Release:	3%{?dist}
+Release:	4%{?dist}
 Summary:	Linux kernel module management utilities
 
 Group:		System Environment/Kernel
@@ -22,6 +22,8 @@ Provides:	module-init-tools = 4.0-1
 Obsoletes:	module-init-tools < 4.0-1
 Provides:	/sbin/modprobe
 
+Patch1:		0001-static-nodes-indicate-that-creation-of-static-nodes-.patch
+
 %description
 The kmod package provides various programs needed for automatic
 loading and unloading of modules under 2.6, 3.x, and later kernels, as well
@@ -48,6 +50,7 @@ applications that wish to load or unload Linux kernel modules.
 
 %prep
 %setup -q
+%patch1 -p1
 
 %build
 export V=1
@@ -111,6 +114,9 @@ install -m 755 %{SOURCE1} $RPM_BUILD_ROOT%{_sbindir}/weak-modules
 %{_libdir}/libkmod.so
 
 %changelog
+* Wed Oct 29 2014 Josh Boyer <jwboyer at fedoraproject.org> - 18-4
+- Backport patch to fix device node permissions (rhbz 1147248)
+
 * Sun Aug 17 2014 Fedora Release Engineering <rel-eng at lists.fedoraproject.org> - 18-3
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_21_22_Mass_Rebuild
 


More information about the scm-commits mailing list