From: Marcos Paulo de Souza <marcos.souza.org@xxxxxxxxx>
Subject: kernel/fork.c: add comment about usage of CLONE_FS flags and namespaces
All other places that deals with namespaces have an explanation of why the
restriction is there. The description added in this commit was based on
e66eded8309eb ("userns: Don't allow CLONE_NEWUSER | CLONE_FS").
Link: http://lkml.kernel.org/r/20171112151637.13258-1-marcos.souza.org@xxxxxxxxx
Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@xxxxxxxxx>
Cc: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
kernel/fork.c | 4 ++++
1 file changed, 4 insertions(+)
diff -puN kernel/fork.c~forkc-add-doc-about-usage-of-clone_fs-flags-and-namespaces kernel/fork.c
--- a/kernel/fork.c~forkc-add-doc-about-usage-of-clone_fs-flags-and-namespaces
+++ a/kernel/fork.c
@@ -1587,6 +1587,10 @@ static __latent_entropy struct task_stru
int retval;
struct task_struct *p;
+ /*
+ * Don't allow sharing the root directory with processes in a different
+ * namespace
+ */
if ((clone_flags & (CLONE_NEWNS|CLONE_FS)) == (CLONE_NEWNS|CLONE_FS))
return ERR_PTR(-EINVAL);
_
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
