You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2015/09/25 21:53:48 UTC

mesos git commit: Fixed a file descriptor leak in the routing code.

Repository: mesos
Updated Branches:
  refs/heads/master 1d44ebac1 -> de9c8dc9a


Fixed a file descriptor leak in the routing code.

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


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

Branch: refs/heads/master
Commit: de9c8dc9a7b3eb3006d914713f5e0dbd9a908825
Parents: 1d44eba
Author: Chi Zhang <ch...@gmail.com>
Authored: Fri Sep 25 12:53:08 2015 -0700
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Fri Sep 25 12:53:09 2015 -0700

----------------------------------------------------------------------
 src/linux/routing/link/link.cpp | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/de9c8dc9/src/linux/routing/link/link.cpp
----------------------------------------------------------------------
diff --git a/src/linux/routing/link/link.cpp b/src/linux/routing/link/link.cpp
index 552e2d7..8ea3e31 100644
--- a/src/linux/routing/link/link.cpp
+++ b/src/linux/routing/link/link.cpp
@@ -374,6 +374,7 @@ Try<bool> setMTU(const string& _link, unsigned int mtu)
     return Error(message);
   }
 
+  os::close(fd);
   return true;
 }