Bug 572706 - udev is creating /dev/snd with the wrong label on MLS
Summary: udev is creating /dev/snd with the wrong label on MLS
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: udev
Version: 13
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Harald Hoyer
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 572707
TreeView+ depends on / blocked
 
Reported: 2010-03-11 21:03 UTC by Daniel Walsh
Modified: 2010-09-26 04:37 UTC (History)
3 users (show)

Fixed In Version: udev-153-4.fc13
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 572707 (view as bug list)
Environment:
Last Closed: 2010-09-26 04:37:02 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
I added this patch and I am not seeing /dev/snd go through this code path. (656 bytes, text/plain)
2010-03-17 14:14 UTC, Daniel Walsh
no flags Details

Description Daniel Walsh 2010-03-11 21:03:11 UTC
Description of problem:

It is creating it with SystemHigh.  (s15:c0.c1023)
system_u:object_r:device_t:s15:c0.c1023

Matchpatcon says it should be s0
system_u:object_r:device_t:s0

This is causing problem on MLS machines.

Comment 1 Harald Hoyer 2010-03-16 10:02:47 UTC
(In reply to comment #0)
> Description of problem:
> 
> It is creating it with SystemHigh.  (s15:c0.c1023)
> system_u:object_r:device_t:s15:c0.c1023
> 
> Matchpatcon says it should be s0
> system_u:object_r:device_t:s0
> 
> This is causing problem on MLS machines.    

udev does not treat any devices with special rules, so it's all in your hand!

Where should udev get the info on how to label the device, if not from selinux itsself?

Comment 2 Daniel Walsh 2010-03-16 14:40:22 UTC
This directory is created as a side effect I believe.

Kernel tells udev to create /dev/snd/seq  and udev creates /dev/snd first.  Does udev ask SELinux what to label this directory or does it just create the directory and then ask how to label the device?  Where in the code is this done?

Comment 3 Harald Hoyer 2010-03-16 16:13:40 UTC
libudev/libudev-util-private.c

util_create_path(..)
        udev_selinux_setfscreatecon(udev, p, S_IFDIR|0755);
        ret = mkdir(p, 0755);
        udev_selinux_resetfscreatecon(udev);

Comment 4 Daniel Walsh 2010-03-17 14:14:13 UTC
Created attachment 400776 [details]
I added this patch and I am not seeing /dev/snd go through this code path.

I am not even sure udev is creating the directory.

Comment 5 Harald Hoyer 2010-03-17 15:10:15 UTC
better change "err" to "info" and set the debuglevel to "info"... err is only compiled, if udev is compiled in debug mode.

Comment 6 Harald Hoyer 2010-04-13 15:50:25 UTC
ping?

Comment 7 Daniel Walsh 2010-04-13 15:56:15 UTC
This is still there although I have no idea why.  I have been working on other stuff but qa is getting around to testing it.

Comment 8 Fedora Update System 2010-09-22 12:09:43 UTC
udev-153-4.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/udev-153-4.fc13

Comment 9 Fedora Update System 2010-09-23 04:58:33 UTC
udev-153-4.fc13 has been pushed to the Fedora 13 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update udev'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/udev-153-4.fc13

Comment 10 Fedora Update System 2010-09-26 04:36:39 UTC
udev-153-4.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.