You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gi...@apache.org on 2019/01/22 19:45:37 UTC

[mesos] branch 1.6.x updated: Fixed a minor coding error in `createSandboxDirectory`.

This is an automated email from the ASF dual-hosted git repository.

gilbert pushed a commit to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/1.6.x by this push:
     new 96fceb0  Fixed a minor coding error in `createSandboxDirectory`.
96fceb0 is described below

commit 96fceb082147def7753a9599ed1a011295a35d8c
Author: Qian Zhang <zh...@gmail.com>
AuthorDate: Tue Jan 22 11:42:49 2019 -0800

    Fixed a minor coding error in `createSandboxDirectory`.
    
    Review: https://reviews.apache.org/r/69805/
    (cherry picked from commit 0335d37fe6905ae84378a418395c38a3de9dfc02)
---
 src/slave/paths.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/slave/paths.cpp b/src/slave/paths.cpp
index ed0b127..dc1003b 100644
--- a/src/slave/paths.cpp
+++ b/src/slave/paths.cpp
@@ -792,7 +792,7 @@ Try<Nothing> createSandboxDirectory(
   // Since this is a sandbox directory containing private task data,
   // we want to ensure that it is not accessible to "others".
   Try<Nothing> chmod = os::chmod(directory, 0750);
-  if (mkdir.isError()) {
+  if (chmod.isError()) {
     return Error("Failed to chmod directory: " + chmod.error());
   }