You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2016/12/12 21:05:40 UTC

[4/4] mesos git commit: Fixed an unreachable statement in launch.cpp.

Fixed an unreachable statement in launch.cpp.

The statement is actually reachable if exec fails. This patch fix it
so that it is actually unreachable.

Review: https://reviews.apache.org/r/54673


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/739e5280
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/739e5280
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/739e5280

Branch: refs/heads/master
Commit: 739e52803e937cd98ef97768ac656ac2059cc722
Parents: 7781e30
Author: Jie Yu <yu...@gmail.com>
Authored: Mon Dec 12 11:20:23 2016 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Mon Dec 12 11:30:31 2016 -0800

----------------------------------------------------------------------
 src/slave/containerizer/mesos/launch.cpp | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/739e5280/src/slave/containerizer/mesos/launch.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/launch.cpp b/src/slave/containerizer/mesos/launch.cpp
index add4ca8..e482ab8 100644
--- a/src/slave/containerizer/mesos/launch.cpp
+++ b/src/slave/containerizer/mesos/launch.cpp
@@ -763,6 +763,7 @@ int MesosContainerizerLaunch::execute()
 
   // If we get here, the execle call failed.
   cerr << "Failed to execute command: " << os::strerror(errno) << endl;
+  exitWithStatus(EXIT_FAILURE);
   UNREACHABLE();
 }