summaryrefslogtreecommitdiff
path: root/namedev.c
diff options
context:
space:
mode:
authorpebenito@gentoo.org <pebenito@gentoo.org>2005-03-08 06:57:25 -0800
committerGreg KH <gregkh@suse.de>2005-04-26 23:39:48 -0700
commitb55e6540260ac3e38d96a0b6c00514a72d5db218 (patch)
treee5ccd55a113138047deabf65824c6f8711e175ff /namedev.c
parente5f053b5312a0f8d62a2bcff8ddb095052d228bc (diff)
[PATCH] udev selinux fix
Here is a fix for the SELinux part of udev. Setfscreatecon() overrides the default labeling behavior of SELinux when creating files, so it should only be used for as short of a time as possible, around the mknod or symlink calls. Without this, the files in udev_db get the wrong label because the fscreatecon is reset after the udev_db file creation instead of before. I'm guessing the Redhat people missed this because they modify udev_db to be one big file instead of a directory of small files (at least that's what I'm told). I created selinux_resetfscreatecon() to reset the fscreatecon asap after the file/node is created. Fixed a memory leak in selinux_init. Getfscreatecon() allocates memory for the context, and the udev code was immediately setting the pointer (security_context_t is actually a typedef'ed char*) to NULL after the call regardless of success/failure. If you're wondering about the case where there's effectively a setfscreatecon(NULL), this is ok, as its used to tell SELinux to do the default labeling behavior. Renamed selinux_restore() to selinux_exit() due to the changed behavior. Fixed a couple of dbg() messages.
Diffstat (limited to 'namedev.c')
0 files changed, 0 insertions, 0 deletions