commit 748fbe5c41d97433dc756a50812e1caf6a6ef727
author Elliott Hughes <enh@google.com> Mon Sep 22 17:43:09 2014 -0700
committer Dmitriy Ivanov <dimitry@google.com> Fri Oct 31 17:03:21 2014 -0700
tree 8b6c5d1bd271d26838e11806c1ec2cb6419a9744
parent d5eb10875affb316c4dfc3b6ceb91df244518956

Fix a couple more cases of missing CLOEXEC.

The debuggerd case can probably never happen, because you're crashing at this
point anyway. The system property one seems possible though.

Bug: 18186310

(cherry picked from commit 0dc39f9952c5e3a3121ea77357bb264ef0f8ded7)

Change-Id: I3e84488fc246f6c28cbd82e96d0cd4343a12c28a

linker/debugger.cpp

diff --git a/linker/debugger.cpp b/linker/debugger.cpp
index 6565985..ac466a5 100644
--- a/linker/debugger.cpp
+++ b/linker/debugger.cpp
@@ -215,7 +215,7 @@
     return;
   }
 
-  int s = socket_abstract_client(DEBUGGER_SOCKET_NAME, SOCK_STREAM);
+  int s = socket_abstract_client(DEBUGGER_SOCKET_NAME, SOCK_STREAM | SOCK_CLOEXEC);
   if (s == -1) {
     __libc_format_log(ANDROID_LOG_FATAL, "libc", "Unable to open connection to debuggerd: %s",
                       strerror(errno));