lkml.org 
[lkml]   [2012]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] fs: print warning when mount flags was ignored
Date
Some mount flags can conflict with each other so they can not be
handled together. Currently when conflicting flags are specified,
some of them are silently ignored putting user in believe that
they was handled correctly.

Fix this by checking for unhandled flags and print warning when
flags was ignored. I am not sure if it's worth translating the flags
into the name, so it just prints the flag as a hexadecimal number.

Obviously we can not fail when conflicting flags are specified,
because it may break buggy application, but it can change in the
future.

Signed-off-by: Lukas Czerner <lczerner@redhat.com>
Reported-by: Milan Broz <mbroz@redhat.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
---
fs/namespace.c | 24 +++++++++++++++++++-----
1 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/fs/namespace.c b/fs/namespace.c
index e608199..8736a48 100644
--- a/fs/namespace.c
+++ b/fs/namespace.c
@@ -2175,18 +2175,32 @@ long do_mount(char *dev_name, char *dir_name, char *type_page,
MS_NOATIME | MS_NODIRATIME | MS_RELATIME| MS_KERNMOUNT |
MS_STRICTATIME);

- if (flags & MS_REMOUNT)
+ if (flags & MS_REMOUNT) {
retval = do_remount(&path, flags & ~MS_REMOUNT, mnt_flags,
data_page);
- else if (flags & MS_BIND)
+ flags &= ~MS_REMOUNT;
+ } else if (flags & MS_BIND) {
retval = do_loopback(&path, dev_name, flags & MS_REC);
- else if (flags & (MS_SHARED | MS_PRIVATE | MS_SLAVE | MS_UNBINDABLE))
+ flags &= ~MS_BIND;
+ } else if (flags & (MS_SHARED | MS_PRIVATE |
+ MS_SLAVE | MS_UNBINDABLE)) {
retval = do_change_type(&path, flags);
- else if (flags & MS_MOVE)
+ flags &= ~(MS_SHARED | MS_PRIVATE | MS_SLAVE | MS_UNBINDABLE);
+ } else if (flags & MS_MOVE) {
retval = do_move_mount(&path, dev_name);
- else
+ flags &= ~MS_MOVE;
+ } else
retval = do_new_mount(&path, type_page, flags, mnt_flags,
dev_name, data_page);
+
+ flags &= (MS_REMOUNT | MS_BIND | MS_SHARED | MS_PRIVATE |
+ MS_SLAVE | MS_UNBINDABLE | MS_MOVE);
+
+ if (!retval && flags)
+ printk(KERN_WARNING "%s(%u): (%s -> %s) Conflicting mount flags"
+ " specified. These flags has been "
+ "ignored: %#.8lx\n", __func__, current->pid,
+ dev_name, dir_name, flags);
dput_out:
path_put(&path);
return retval;
--
1.7.4.4


\
 
 \ /
  Last update: 2012-04-19 11:11    [W:2.514 / U:0.808 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site