]> git.baikalelectronics.ru Git - kernel.git/commit
fs/exec: allow to unshare a time namespace on vfork+exec
authorAndrei Vagin <avagin@gmail.com>
Mon, 13 Jun 2022 06:07:22 +0000 (23:07 -0700)
committerKees Cook <keescook@chromium.org>
Wed, 15 Jun 2022 14:58:04 +0000 (07:58 -0700)
commit6718a0918b49354e792e8beb7e432d59ea03c65f
tree04ff6fdda3b663c9069512108da2b73483134af8
parent71374b58ebc7666369d670c06d1176f81c437c59
fs/exec: allow to unshare a time namespace on vfork+exec

Right now, a new process can't be forked in another time namespace
if it shares mm with its parent. It is prohibited, because each time
namespace has its own vvar page that is mapped into a process address
space.

When a process calls exec, it gets a new mm and so it could be "legal"
to switch time namespace in that case. This was not implemented and
now if we want to do this, we need to add another clone flag to not
break backward compatibility.

We don't have any user requests to switch times on exec except the
vfork+exec combination, so there is no reason to add a new clone flag.
As for vfork+exec, this should be safe to allow switching timens with
the current clone flag. Right now, vfork (CLONE_VFORK | CLONE_VM) fails
if a child is forked into another time namespace. With this change,
vfork creates a new process in parent's timens, and the following exec
does the actual switch to the target time namespace.

Suggested-by: Florian Weimer <fweimer@redhat.com>
Signed-off-by: Andrei Vagin <avagin@gmail.com>
Acked-by: Christian Brauner (Microsoft) <brauner@kernel.org>
Signed-off-by: Kees Cook <keescook@chromium.org>
Link: https://lore.kernel.org/r/20220613060723.197407-1-avagin@gmail.com
fs/exec.c
kernel/fork.c
kernel/nsproxy.c