From 0ebbd3caefbd7b267ab149a4e98e64e8f9812797 Mon Sep 17 00:00:00 2001 From: Harshavardhana Date: Wed, 3 Jul 2019 18:57:34 -0700 Subject: [PATCH] Avoid chown instead fallback to rootpath for user perms (#7874) Fixes #7864 --- dockerscripts/check-user.go | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/dockerscripts/check-user.go b/dockerscripts/check-user.go index 00fe03f8a..8b915ad8d 100644 --- a/dockerscripts/check-user.go +++ b/dockerscripts/check-user.go @@ -22,7 +22,6 @@ import ( "fmt" "log" "os" - "os/exec" "os/user" "syscall" @@ -43,13 +42,13 @@ func getUserGroup(path string) (string, error) { if err != nil { // Fresh directory we should default to what was requested by user. if os.IsNotExist(err) { - cmd := exec.Command("chown", "-R", defaultUserGroup, path) - if err = cmd.Run(); err != nil { + fi, err = os.Stat(path) + if err != nil { return "", err } - return defaultUserGroup, nil + } else { + return "", err } - return "", err } stat, ok := fi.Sys().(*syscall.Stat_t) if !ok {