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 2014/10/17 19:32:01 UTC
git commit: Fixed a file descriptor leak in StatusUpdateManager.
Repository: mesos
Updated Branches:
refs/heads/master 48260b7d7 -> 30d12fa1e
Fixed a file descriptor leak in StatusUpdateManager.
Review: https://reviews.apache.org/r/26879
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/30d12fa1
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/30d12fa1
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/30d12fa1
Branch: refs/heads/master
Commit: 30d12fa1e3d5b9e4e9aef3a720ce68e7573c382c
Parents: 48260b7
Author: Jie Yu <yu...@gmail.com>
Authored: Fri Oct 17 10:21:04 2014 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Fri Oct 17 10:21:17 2014 -0700
----------------------------------------------------------------------
src/slave/status_update_manager.hpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/30d12fa1/src/slave/status_update_manager.hpp
----------------------------------------------------------------------
diff --git a/src/slave/status_update_manager.hpp b/src/slave/status_update_manager.hpp
index 24e3882..c371e55 100644
--- a/src/slave/status_update_manager.hpp
+++ b/src/slave/status_update_manager.hpp
@@ -179,7 +179,7 @@ struct StatusUpdateStream
// Open the updates file.
Try<int> result = os::open(
path.get(),
- O_CREAT | O_WRONLY | O_APPEND | O_SYNC,
+ O_CREAT | O_WRONLY | O_APPEND | O_SYNC | O_CLOEXEC,
S_IRUSR | S_IWUSR | S_IRGRP | S_IRWXO);
if (result.isError()) {