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.

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

need fexecve #88

Closed
gdamore opened this Issue Nov 1, 2014 · 3 comments

Comments

Projects
None yet
1 participant
@gdamore
Owner

gdamore commented Nov 1, 2014

POSIX 2008 / XPG7 require fexecve(), which provides a file descriptor as its first argument, instead of a path.

Note the following considerations:

  • While Linux uses /proc/$$/fd/, we cannot -- because /proc grabs locks, and attempts to make this work result in deadlock.
  • So a syscall looks best
  • However, we can write /proc/$$/fd/ in the uarea as the system image. This will facilitate post mortem analysis in the event of a crash.
  • We should take care not to exec() the file appears to be in extended attribute space.

@gdamore

This comment has been minimized.

Show comment
Hide comment
@gdamore

gdamore Nov 1, 2014

Owner

fixes #88 need fexecve (fix lint)

→ <<cset 20189e4>>


Original Comment By: Garrett D'Amore

Owner

gdamore commented Nov 1, 2014

fixes #88 need fexecve (fix lint)

→ <<cset 20189e4>>


Original Comment By: Garrett D'Amore

@gdamore

This comment has been minimized.

Show comment
Hide comment
@gdamore

gdamore Nov 1, 2014

Owner

fixes #88 need fexecve (codereview feedback)

→ <<cset 0815dd5>>


Original Comment By: Garrett D'Amore

Owner

gdamore commented Nov 1, 2014

fixes #88 need fexecve (codereview feedback)

→ <<cset 0815dd5>>


Original Comment By: Garrett D'Amore

@gdamore

This comment has been minimized.

Show comment
Hide comment
@gdamore

gdamore Nov 1, 2014

Owner

fixes #88 need fexecve

→ <<cset 10f26e9>>


Original Comment By: Garrett D'Amore

Owner

gdamore commented Nov 1, 2014

fixes #88 need fexecve

→ <<cset 10f26e9>>


Original Comment By: Garrett D'Amore

@gdamore gdamore closed this Nov 1, 2014

gdamore added a commit that referenced this issue Nov 1, 2014

gdamore added a commit that referenced this issue Nov 1, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment