Set close on exec for client socket

To close the client socket fd, this patch sets O_CLOEXEC flag using fcntl().

Change-Id: I90f60c23ffc8aa0e62c93e26b4494493fb08eedb
Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>

diff --git a/src/client-socket-internal.cc b/src/client-socket-internal.cc
index 4fbb3449f298ce22d2b4a3152ca699f4bf23143d..9b0a1072f5d80ace6260c1e913cd2dfbaf052f00 100644 (file)
--- a/src/client-socket-internal.cc
+++ b/src/client-socket-internal.cc
@@ -39,6 +39,7 @@ ClientSocket::ClientSocket() {
 }
 
 ClientSocket::ClientSocket(int fd) : fd_(fd) {
+  SetCloseOnExec();
 }
 
 ClientSocket::~ClientSocket() {
@@ -46,6 +47,12 @@ ClientSocket::~ClientSocket() {
     Close();
 }
 
+void ClientSocket::SetCloseOnExec() {
+  int flags = fcntl(fd_, F_GETFL, 0);
+  fcntl(fd_, F_SETFL, flags | O_CLOEXEC);
+  _I("Close on exec. fd(%d)", fd_);
+}
+
 void ClientSocket::Close() {
   if (fd_ > -1) {
     close(fd_);

diff --git a/src/client-socket-internal.hh b/src/client-socket-internal.hh
index 2446e389b1f60e822fbb70e94693351b09fc0feb..772b45245afd26dc7d98e49232c59e8245ca1e8d 100644 (file)
--- a/src/client-socket-internal.hh
+++ b/src/client-socket-internal.hh
@@ -42,6 +42,7 @@ class ClientSocket {
   int GetFd() const;
   int RemoveFd();
   void SetNonblock();
+  void SetCloseOnExec();
 
  private:
   int fd_;

diff --git a/src/server-socket-internal.cc b/src/server-socket-internal.cc
index 9f826405ab7249b92cc1b635857b2b1d83c2de5e..9fd7e4323d35bdeaa3ba39dd9296cb7fc1356095 100644 (file)
--- a/src/server-socket-internal.cc
+++ b/src/server-socket-internal.cc
@@ -71,7 +71,7 @@ int ServerSocket::Listen(int backlog) {
 void ServerSocket::SetCloseOnExec() {
   int flags = fcntl(fd_, F_GETFL, 0);
   fcntl(fd_, F_SETFL, flags | O_CLOEXEC);
-  _I("Close on exec");
+  _I("Close on exec. fd(%d)", fd_);
 }
 
 void ServerSocket::Close() {