diff options
author | Giuseppe Scrivano <gscrivan@redhat.com> | 2018-08-29 10:01:45 +0200 |
---|---|---|
committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-08-29 16:25:20 +0000 |
commit | 1789242933ddbc3e4a29662f5218b5b94ee30863 (patch) | |
tree | cf9a0b2106f66446bef638fb1fc7860fd27c2972 | |
parent | eb5fdebc84e59f8645218f8fdfd6e6a8136cfa43 (diff) | |
download | podman-1789242933ddbc3e4a29662f5218b5b94ee30863.tar.gz podman-1789242933ddbc3e4a29662f5218b5b94ee30863.tar.bz2 podman-1789242933ddbc3e4a29662f5218b5b94ee30863.zip |
rootless: add new function to join existing namespace
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Closes: #1371
Approved by: rhatdan
-rw-r--r-- | pkg/rootless/rootless_linux.c | 32 | ||||
-rw-r--r-- | pkg/rootless/rootless_linux.go | 29 | ||||
-rw-r--r-- | pkg/rootless/rootless_unsupported.go | 6 |
3 files changed, 65 insertions, 2 deletions
diff --git a/pkg/rootless/rootless_linux.c b/pkg/rootless/rootless_linux.c index 1b98f76eb..e894328ce 100644 --- a/pkg/rootless/rootless_linux.c +++ b/pkg/rootless/rootless_linux.c @@ -83,7 +83,37 @@ get_cmd_line_args (pid_t pid) } int -reexec_in_user_namespace(int ready) +reexec_userns_join (int userns) +{ + pid_t ppid = getpid (); + char uid[16]; + char **argv; + int pid; + + sprintf (uid, "%d", geteuid ()); + + argv = get_cmd_line_args (ppid); + if (argv == NULL) + _exit (EXIT_FAILURE); + + pid = fork (); + if (pid) + return pid; + + setenv ("_LIBPOD_USERNS_CONFIGURED", "init", 1); + setenv ("_LIBPOD_ROOTLESS_UID", uid, 1); + + if (setns (userns, 0) < 0) + _exit (EXIT_FAILURE); + close (userns); + + execvp (argv[0], argv); + + _exit (EXIT_FAILURE); +} + +int +reexec_in_user_namespace (int ready) { int ret; pid_t pid; diff --git a/pkg/rootless/rootless_linux.go b/pkg/rootless/rootless_linux.go index 904d22ee2..e218dede0 100644 --- a/pkg/rootless/rootless_linux.go +++ b/pkg/rootless/rootless_linux.go @@ -23,6 +23,7 @@ import ( /* extern int reexec_in_user_namespace(int ready); extern int reexec_in_user_namespace_wait(int pid); +extern int reexec_userns_join(int userns); */ import "C" @@ -84,6 +85,32 @@ func tryMappingTool(tool string, pid int, hostID int, mappings []idtools.IDMap) return cmd.Run() } +// JoinNS re-exec podman in a new userNS and join the user namespace of the specified +// PID. +func JoinNS(pid uint) (bool, int, error) { + if os.Getuid() == 0 || os.Getenv("_LIBPOD_USERNS_CONFIGURED") != "" { + return false, -1, nil + } + + userNS, err := GetUserNSForPid(pid) + if err != nil { + return false, -1, err + } + defer userNS.Close() + + pidC := C.reexec_userns_join(C.int(userNS.Fd())) + if int(pidC) < 0 { + return false, -1, errors.Errorf("cannot re-exec process") + } + + ret := C.reexec_in_user_namespace_wait(pidC) + if ret < 0 { + return false, -1, errors.New("error waiting for the re-exec process") + } + + return true, int(ret), nil +} + // BecomeRootInUserNS re-exec podman in a new userNS. It returns whether podman was re-executed // into a new user namespace and the return code from the re-executed podman process. // If podman was re-executed the caller needs to propagate the error code returned by the child @@ -183,7 +210,7 @@ func BecomeRootInUserNS() (bool, int, error) { ret := C.reexec_in_user_namespace_wait(pidC) if ret < 0 { - return false, -1, errors.Wrapf(err, "error waiting for the re-exec process") + return false, -1, errors.New("error waiting for the re-exec process") } return true, int(ret), nil diff --git a/pkg/rootless/rootless_unsupported.go b/pkg/rootless/rootless_unsupported.go index 93b04adfd..e0adb3453 100644 --- a/pkg/rootless/rootless_unsupported.go +++ b/pkg/rootless/rootless_unsupported.go @@ -33,6 +33,12 @@ func SkipStorageSetup() bool { return false } +// JoinNS re-exec podman in a new userNS and join the user namespace of the specified +// PID. +func JoinNS(pid uint) (bool, int, error) { + return false, -1, errors.New("this function is not supported on this os") +} + // GetUserNSForPid returns an open FD for the first direct child user namespace that created the process func GetUserNSForPid(pid uint) (*os.File, error) { return nil, errors.New("this function is not supported on this os") |