diff options
author | Eric Sandeen <sandeen@redhat.com> | 2008-02-06 04:38:34 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-06 13:41:12 -0500 |
commit | 2830bfd6cf66133c86d4a32004fd99c3de7e23bf (patch) | |
tree | 51f68453ad6b5a3a75667385369bbd9b5ba8a02c /fs/ecryptfs/super.c | |
parent | 99db6e4a9764887842006a2b1aa804de6171db42 (diff) |
ecryptfs: remove debug as mount option, and warn if set via modprobe
ecryptfs_debug really should not be a mount option; it is not per-mount,
but rather sets a global "ecryptfs_verbosity" variable which affects all
mounted filesysytems. It's already settable as a module load option,
I think we can leave it at that.
Also, if set, since secret values come out in debug messages, kick
things off with a stern warning.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Acked-by: Mike Halcrow <mhalcrow@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ecryptfs/super.c')
-rw-r--r-- | fs/ecryptfs/super.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/ecryptfs/super.c b/fs/ecryptfs/super.c index 0556604e8dc2..c27ac2b358a1 100644 --- a/fs/ecryptfs/super.c +++ b/fs/ecryptfs/super.c | |||
@@ -174,10 +174,6 @@ static int ecryptfs_show_options(struct seq_file *m, struct vfsmount *mnt) | |||
174 | } | 174 | } |
175 | mutex_unlock(&mount_crypt_stat->global_auth_tok_list_mutex); | 175 | mutex_unlock(&mount_crypt_stat->global_auth_tok_list_mutex); |
176 | 176 | ||
177 | /* Note this is global and probably shouldn't be a mount option */ | ||
178 | if (ecryptfs_verbosity) | ||
179 | seq_printf(m, ",ecryptfs_debug=%d\n", ecryptfs_verbosity); | ||
180 | |||
181 | seq_printf(m, ",ecryptfs_cipher=%s", | 177 | seq_printf(m, ",ecryptfs_cipher=%s", |
182 | mount_crypt_stat->global_default_cipher_name); | 178 | mount_crypt_stat->global_default_cipher_name); |
183 | 179 | ||