luxolus/open-nomad
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

executor_linux only do path resolution in the taskDir, not local

Browse source 
split out lookPathIn to show it's similarity to exec.LookPath
This commit is contained in:
Lang Martin 2019-05-10 11:33:35 -04:00
parent 3ae276cfd2
commit 99359d7fbe
1 changed files with 23 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
drivers/shared/executor/executor_linux.go
View file

@ -791,6 +791,10 @@ func lookupTaskBin(command *ExecCommand) (string, error) {
return root, nil
}
if strings.Contains(bin, "/") {
return "", fmt.Errorf("file %s not found under path %s", bin, taskDir)
}
// Find the PATH
path := "/usr/local/bin:/usr/bin:/bin"
for _, e := range command.Env {
@ -799,14 +803,17 @@ func lookupTaskBin(command *ExecCommand) (string, error) {
}
}
// Check the host PATH anchored inside the taskDir
if !strings.Contains(bin, "/") {
return lookPathIn(path, taskDir, bin)
}
// lookPathIn looks for a file with PATH inside the directory root. Like exec.LookPath
func lookPathIn(path string, root string, bin string) (string, error) {
// exec.LookPath(file string)
for _, dir := range filepath.SplitList(path) {
if dir == "" {
// match unix shell behavior, empty path element == .
dir = "."
}
for _, root := range []string{localDir, taskDir} {
path := filepath.Join(root, dir, bin)
f, err := os.Stat(path)
if err != nil {
@ -816,8 +823,5 @@ func lookupTaskBin(command *ExecCommand) (string, error) {
return path, nil
}
}
}
}
return "", fmt.Errorf("file %s not found under path %s", bin, taskDir)
return "", fmt.Errorf("file %s not found under path %s", bin, root)
}