Repository: mesos Updated Branches: refs/heads/master 5a7f66e13 -> 93c39b89e
Updated subprocess to use os::execvpe. Review: https://reviews.apache.org/r/24681 Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/49e94f3a Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/49e94f3a Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/49e94f3a Branch: refs/heads/master Commit: 49e94f3abbc9a04cc8896f61e55975fbae5d888e Parents: bc5ab04 Author: Jie Yu <yujie....@gmail.com> Authored: Wed Aug 13 16:26:28 2014 -0700 Committer: Jie Yu <yujie....@gmail.com> Committed: Wed Aug 13 20:52:34 2014 -0700 ---------------------------------------------------------------------- 3rdparty/libprocess/src/subprocess.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/49e94f3a/3rdparty/libprocess/src/subprocess.cpp ---------------------------------------------------------------------- diff --git a/3rdparty/libprocess/src/subprocess.cpp b/3rdparty/libprocess/src/subprocess.cpp index 68bfd5d..5b92200 100644 --- a/3rdparty/libprocess/src/subprocess.cpp +++ b/3rdparty/libprocess/src/subprocess.cpp @@ -160,9 +160,9 @@ static int childMain( } } - execve(path.c_str(), argv, (*envp)()); + os::execvpe(path.c_str(), argv, (*envp)()); - ABORT("Failed to execve in childMain\n"); + ABORT("Failed to os::execvpe in childMain\n"); return UNREACHABLE(); } @@ -315,8 +315,8 @@ Try<Subprocess> subprocess( } } - // The real arguments that will be passed to 'execve'. We need to - // construct them here before doing the clone as it might not be + // The real arguments that will be passed to 'os::execvpe'. We need + // to construct them here before doing the clone as it might not be // async signal safe. char** _argv = new char*[argv.size() + 1]; for (int i = 0; i < argv.size(); i++) {