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());
}