[PATCH] rules: set the correct group of the static snd/{seq,timer} nodes

[PATCH] rules: set the correct group of the static snd/{seq,timer} nodes

[Tom Gundersen]

From 01b159b3f67d5e387ebab879e52ae2d1b6be32bf Mon Sep 17 00:00:00 2001
From: Tom Gundersen <teg@jklm.no>
Date: Tue, 24 May 2011 15:04:56 +0200
Subject: [PATCH] rules: set the correct group of the static
snd/{seq,timer} nodes

When removing some of our custom rules from Arch we found that
a user in the audio group could no longer autoload the snd_seq
module. This fixes it, as proposed by Kay.

An analogous fix is applied to snd/timer.

Signed-off-by: Tom Gundersen <teg@jklm.no>
---
 rules/rules.d/50-udev-default.rules |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/rules/rules.d/50-udev-default.rules
b/rules/rules.d/50-udev-default.rules
index cd745ef..f7f134b 100644
--- a/rules/rules.d/50-udev-default.rules
+++ b/rules/rules.d/50-udev-default.rules
@@ -39,6 +39,10 @@ SUBSYSTEM="drm",		GROUP="video"

 # sound
 SUBSYSTEM="sound",		GROUP="audio"
+KERNEL="seq",			GROUP="audio", MODE="0660", \
+  OPTIONS+="static_node=snd/seq"
+KERNEL="timer",		GROUP="audio", MODE="0660", \
+  OPTIONS+="static_node=snd/timer"

 # DVB (video)
 SUBSYSTEM="dvb", GROUP="video"
-- 
1.7.5.2

Re: [PATCH] rules: set the correct group of the static

[Kay Sievers]

On Tue, May 24, 2011 at 15:33, Tom Gundersen <teg@jklm.no> wrote:
> When removing some of our custom rules from Arch we found that
> a user in the audio group could no longer autoload the snd_seq
> module. This fixes it, as proposed by Kay.

Found out that we can use something simpler. Should work now.

Thanks for the reminder,
Kay

Re: [PATCH] rules: set the correct group of the static

[Tom Gundersen]

[forgot to cc]
On Fri, May 27, 2011 at 2:52 AM, Kay Sievers <kay.sievers@vrfy.org> wrote:
> On Tue, May 24, 2011 at 15:33, Tom Gundersen <teg@jklm.no> wrote:
>> When removing some of our custom rules from Arch we found that
>> a user in the audio group could no longer autoload the snd_seq
>> module. This fixes it, as proposed by Kay.
>
> Found out that we can use something simpler. Should work now.

Close, but no cigar. Looks like the static nodes are not assigned
permissions 0660 even if a gid is set (the nodes have perms 0600).

Cheers,

Tom

Re: [PATCH] rules: set the correct group of the static

[Kay Sievers]

On Mon, May 30, 2011 at 01:35, Tom Gundersen <teg@jklm.no> wrote:
> On Fri, May 27, 2011 at 2:52 AM, Kay Sievers <kay.sievers@vrfy.org> wrote:
>> On Tue, May 24, 2011 at 15:33, Tom Gundersen <teg@jklm.no> wrote:
>>> When removing some of our custom rules from Arch we found that
>>> a user in the audio group could no longer autoload the snd_seq
>>> module. This fixes it, as proposed by Kay.
>>
>> Found out that we can use something simpler. Should work now.
>
> Close, but no cigar. Looks like the static nodes are not assigned
> permissions 0660 even if a gid is set (the nodes have perms 0600).

rules: static_node - use 0660 if group is given to get the cigar
http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h¡12873b5bc9ebbae39c32f502bc6211f33546cc

Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] rules: set the correct group of the static

[Tom Gundersen]

On Mon, May 30, 2011 at 2:27 AM, Kay Sievers <kay.sievers@vrfy.org> wrote:
> On Mon, May 30, 2011 at 01:35, Tom Gundersen <teg@jklm.no> wrote:
>> On Fri, May 27, 2011 at 2:52 AM, Kay Sievers <kay.sievers@vrfy.org> wrote:
>>> On Tue, May 24, 2011 at 15:33, Tom Gundersen <teg@jklm.no> wrote:
>>>> When removing some of our custom rules from Arch we found that
>>>> a user in the audio group could no longer autoload the snd_seq
>>>> module. This fixes it, as proposed by Kay.
>>>
>>> Found out that we can use something simpler. Should work now.
>>
>> Close, but no cigar. Looks like the static nodes are not assigned
>> permissions 0660 even if a gid is set (the nodes have perms 0600).
>
> rules: static_node - use 0660 if group is given to get the cigar
> http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h¡12873b5bc9ebbae39c32f502bc6211f33546cc

Thanks!

-t
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html