diff options
| author | Jim Meyering <meyering@redhat.com> | 2009-10-07 15:31:17 +0200 |
|---|---|---|
| committer | Jim Meyering <meyering@redhat.com> | 2009-10-09 14:33:10 +0200 |
| commit | 0023f65fd7a2ba53b0209c7b1a66c5c30612d0c2 (patch) | |
| tree | f399cfe56474a9b50e9382c903cdd02863002417 | |
| parent | 57d9263ca54a35e76fc8ed2ec3ec8dc36f8da60b (diff) | |
| download | coreutils-0023f65fd7a2ba53b0209c7b1a66c5c30612d0c2.tar.xz | |
chcon: don't disable just because SELinux is disabled
* src/chcon.c (main): Now that gnulib provides getfilecon wrappers,
we can revert most of the 2009-10-05 commit 3a97d664, "chcon: exit
immediately if SELinux is disabled", since chcon is still useful as
long as the file system provides handlers for the security.*
name space. gnulib's getfilecon wrappers ensure that an offending
context now evokes a return value of -1.
Prompted by comments from Stephen Smalley in
http://thread.gmane.org/gmane.comp.gnu.coreutils.bugs/18378/focus=18394
* NEWS (Bug fixes): Mention it.
| -rw-r--r-- | NEWS | 4 | ||||
| -rw-r--r-- | src/chcon.c | 4 |
2 files changed, 4 insertions, 4 deletions
@@ -4,6 +4,10 @@ GNU coreutils NEWS -*- outline -*- ** Bug fixes + chcon no longer exits immediately just because SELinux is disabled. + Even then, chcon may still be useful. + [bug introduced in coreutils-8.0] + stat -f recognizes more file system types: afs, cifs, anon-inode FS, btrfs, cgroupfs, cramfs-wend, debugfs, futexfs, hfs, inotifyfs, minux3, nilfs, securityfs, selinux, xenfs diff --git a/src/chcon.c b/src/chcon.c index c0da6949a..fbfdb4d29 100644 --- a/src/chcon.c +++ b/src/chcon.c @@ -519,10 +519,6 @@ main (int argc, char **argv) usage (EXIT_FAILURE); } - if (is_selinux_enabled () != 1) - error (EXIT_FAILURE, 0, - _("%s may be used only on a SELinux kernel"), program_name); - if (reference_file) { if (getfilecon (reference_file, &ref_context) < 0) |