diff options
| author | Giuseppe Scrivano <gscrivan@redhat.com> | 2018-08-31 09:31:34 +0200 |
|---|---|---|
| committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-09-04 14:36:57 +0000 |
| commit | 807f6f8d8f98422cfcfe7e474e26a985d951af4d (patch) | |
| tree | 6010ce20c6ce8f247cbb8b3e6f29c294521c82ad /cmd/podman/create.go | |
| parent | bdee681409f595443d4f3cb4f08b76d24139d6a8 (diff) | |
| download | podman-807f6f8d8f98422cfcfe7e474e26a985d951af4d.tar.gz podman-807f6f8d8f98422cfcfe7e474e26a985d951af4d.tar.bz2 podman-807f6f8d8f98422cfcfe7e474e26a985d951af4d.zip | |
rootless: check uid with Geteuid() instead of Getuid()
change the tests to use chroot to set a numeric UID/GID.
Go syscall.Credential doesn't change the effective UID/GID of the
process.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Closes: #1372
Approved by: mheon
Diffstat (limited to 'cmd/podman/create.go')
| -rw-r--r-- | cmd/podman/create.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cmd/podman/create.go b/cmd/podman/create.go index 04f3cd9e6..20d976c0b 100644 --- a/cmd/podman/create.go +++ b/cmd/podman/create.go @@ -99,7 +99,7 @@ func createCmd(c *cli.Context) error { storageOpts.UIDMap = mappings.UIDMap storageOpts.GIDMap = mappings.GIDMap - if os.Getuid() != 0 { + if os.Geteuid() != 0 { rootless.SetSkipStorageSetup(true) } @@ -778,7 +778,7 @@ func parseCreateOpts(ctx context.Context, c *cli.Context, runtime *libpod.Runtim } func joinOrCreateRootlessUserNamespace(createConfig *cc.CreateConfig, runtime *libpod.Runtime) (bool, int, error) { - if os.Getuid() == 0 { + if os.Geteuid() == 0 { return false, 0, nil } |