]> err.no Git - util-linux/commitdiff
mount: make file_t SELinux warning optional and shorter
authorKarel Zak <kzak@redhat.com>
Wed, 23 Jul 2008 23:20:00 +0000 (01:20 +0200)
committerKarel Zak <kzak@redhat.com>
Fri, 12 Dec 2008 12:29:12 +0000 (13:29 +0100)
The patch:

   Commit:  4edebc1486133231e38b3881325c374eda567f74
   Date:    Mon Jun 23 13:00:00 2008 +0200
   Subject: mount: warn on "file_t" selinux context

introduces a new warning. This warning should be shorter and optional
-- it means visible in verbose mode only.

Signed-off-by: Karel Zak <kzak@redhat.com>
mount/mount.c

index 0d28a51807a3555869bf85616617782a148e9258..9dc3263ef501db87703cb796b9cee386faa10f2c 100644 (file)
@@ -1376,7 +1376,7 @@ try_mount_one (const char *spec0, const char *node0, const char *types0,
  out:
 
 #ifdef HAVE_LIBSELINUX
-  if (res != EX_FAIL && is_selinux_enabled() > 0) {
+  if (res != EX_FAIL && verbose && is_selinux_enabled() > 0) {
       security_context_t raw = NULL, def = NULL;
 
       if (getfilecon(node, &raw) > 0 &&
@@ -1387,10 +1387,8 @@ try_mount_one (const char *spec0, const char *node0, const char *types0,
                    "       You just mounted an file system that supports labels which does not\n"
                    "       contain labels, onto an SELinux box. It is likely that confined\n"
                    "       applications will generate AVC messages and not be allowed access to\n"
-                   "       this file system.  You can add labels to this file system by executing\n"
-                   "       restorecon(8). If you do not want to add labels to this file system,\n"
-                   "       you should mount the file system using one of the \"context\" mount\n"
-                   "       option."), node);
+                   "       this file system.  For more details see restorecon(8) and mount(8).\n"),
+                   node);
       }
       freecon(raw);
       freecon(def);