[Devel,RHEL7,COMMIT] ve/proc/modules: show loaded modules in container

Submitted by Konstantin Khorenko on Nov. 23, 2016, 3:48 p.m.

Details

Message ID 201611231548.uANFmG1q011083@finist_cl7.x64_64.work.ct
State New
Series "proc/modules: show loaded modules in container."
Headers show

Commit Message

Konstantin Khorenko Nov. 23, 2016, 3:48 p.m.
The commit is pushed to "branch-rh7-3.10.0-327.36.1.vz7.20.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-327.36.1.vz7.20.1
------>
commit 08312c794cade2d0cec1c367cbf0aaafba78243e
Author: Andrey Ryabinin <aryabinin@virtuozzo.com>
Date:   Wed Nov 23 19:48:15 2016 +0400

    ve/proc/modules: show loaded modules in container
    
    Some scripts wants to read this file to perform some sanity checks.
    Loaded modules is not security sensitive information, so this shouldn't
    be a problem to expose this in container.
    
    https://jira.sw.ru/browse/PSBM-55920
    https://bugs.openvz.org/browse/OVZ-6834
    
    Signed-off-by: Andrey Ryabinin <aryabinin@virtuozzo.com>
---
 kernel/module.c | 2 --
 1 file changed, 2 deletions(-)

Patch hide | download patch | download mbox

diff --git a/kernel/module.c b/kernel/module.c
index 762716f..931987c 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -3742,8 +3742,6 @@  static char *module_flags(struct module *mod, char *buf)
 static void *m_start(struct seq_file *m, loff_t *pos)
 {
 	mutex_lock(&module_mutex);
-	if (!ve_is_super(get_exec_env()))
-		return NULL;
 	return seq_list_start(&modules, *pos);
 }