Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

$PATH lookup before exec'ing the container process #13

Closed
giuseppe opened this issue Feb 22, 2019 · 0 comments · Fixed by #73
Closed

$PATH lookup before exec'ing the container process #13

giuseppe opened this issue Feb 22, 2019 · 0 comments · Fixed by #73
Labels
enhancement good first issue help wanted

Comments

@giuseppe
Copy link
Member

to be compatible with runC, we need to lookup argv[0] into $PATH once we are in the new mount namespace, so that start can fail immediately if the specified command doesn't exist.
giuseppe added a commit to giuseppe/crun that referenced this issue Aug 14, 2019
@giuseppe
container: add executable lookup
e867485 
Closes: containers#13

Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
@giuseppe giuseppe mentioned this issue Aug 14, 2019
Merged
giuseppe added a commit to giuseppe/crun that referenced this issue Aug 14, 2019
@giuseppe
container: add executable lookup
16cc131 
Closes: containers#13

Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
giuseppe added a commit to giuseppe/crun that referenced this issue Aug 14, 2019
@giuseppe
container: add executable lookup
573f531 
Closes: containers#13

Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
giuseppe added a commit to giuseppe/crun that referenced this issue Aug 14, 2019
@giuseppe
container: add executable lookup
fc12666 
Closes: containers#13

Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
giuseppe added a commit to giuseppe/crun that referenced this issue Aug 14, 2019
@giuseppe
container: add executable lookup
c047a49 
Closes: containers#13

Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

container: add executable lookup giuseppe/crun
1 participant
@giuseppe