Ecore_Conn: Enable CLOEXEC by default.

This flag should be enabled by default in order to avoid socket leaks.

5 files changed, 3 insertions, 7 deletions

diff --git a/src/lib/ecore_con/ecore_con_legacy.c b/src/lib/ecore_con/ecore_con_legacy.c
index 35cdef87cd..1cb007f8b1 100644
--- a/src/lib/ecore_con/ecore_con_legacy.c
+++ b/src/lib/ecore_con/ecore_con_legacy.c
@@ -1496,11 +1496,9 @@ _ecore_con_server_server_set(Ecore_Con_Server *svr, Eo *server)
 
    if (efl_isa(inner_server, EFL_NET_SERVER_FD_CLASS))
      {
-        efl_net_server_fd_close_on_exec_set(inner_server, EINA_TRUE);
         efl_net_server_fd_reuse_address_set(inner_server, EINA_TRUE);
         efl_net_server_fd_reuse_port_set(inner_server, EINA_TRUE);
      }
-
    if (efl_isa(inner_server, EFL_NET_SERVER_TCP_CLASS))
      {
         /* old ecore_con did not map ipv4 to ipv6... */
@@ -1516,7 +1514,6 @@ _ecore_con_server_server_set(Ecore_Con_Server *svr, Eo *server)
      {
         /* old ecore_con did not map ipv4 to ipv6... */
         efl_net_server_ssl_ipv6_only_set(inner_server, EINA_TRUE);
-        efl_net_server_ssl_close_on_exec_set(inner_server, EINA_TRUE);
         efl_net_server_ssl_reuse_address_set(inner_server, EINA_TRUE);
         efl_net_server_ssl_reuse_port_set(inner_server, EINA_TRUE);
      }
diff --git a/src/lib/ecore_con/efl_net_server_fd.c b/src/lib/ecore_con/efl_net_server_fd.c
index 03448ec856..f1ff38bd77 100644
--- a/src/lib/ecore_con/efl_net_server_fd.c
+++ b/src/lib/ecore_con/efl_net_server_fd.c
@@ -100,6 +100,7 @@ EOLIAN static Efl_Object *
 _efl_net_server_fd_efl_object_constructor(Eo *o, Efl_Net_Server_Fd_Data *pd)
 {
    pd->family = AF_UNSPEC;
+   pd->close_on_exec = EINA_TRUE;
    return efl_constructor(efl_super(o, MY_CLASS));
 }
 
diff --git a/src/lib/ecore_con/efl_net_server_fd.eo b/src/lib/ecore_con/efl_net_server_fd.eo
index 0b3f55df98..6f3ab66332 100644
--- a/src/lib/ecore_con/efl_net_server_fd.eo
+++ b/src/lib/ecore_con/efl_net_server_fd.eo
@@ -71,7 +71,7 @@ class Efl.Net.Server.Fd (Efl.Loop.Fd, Efl.Net.Server) {
         @property close_on_exec {
             [[Controls Close-on-Exec() using FD_CLOEXEC.
 
-              Children socket will inherit the server's setting by
+              Children socket will not inherit the server's setting by
               default. One can change the behavior using each instance
               @Efl.Io.Closer.close_on_exec.set.
             ]]
diff --git a/src/lib/ecore_con/efl_net_server_ssl.eo b/src/lib/ecore_con/efl_net_server_ssl.eo
index d34e2df909..594104d1b7 100644
--- a/src/lib/ecore_con/efl_net_server_ssl.eo
+++ b/src/lib/ecore_con/efl_net_server_ssl.eo
@@ -71,7 +71,7 @@ class Efl.Net.Server.Ssl (Efl.Loop_User, Efl.Net.Server) {
         @property close_on_exec {
             [[Controls Close-on-Exec() using FD_CLOEXEC.
 
-              Children socket will inherit the server's setting by
+              Children socket will not inherit the server's setting by
               default. One can change the behavior using each instance
               @Efl.Io.Closer.close_on_exec.set.
             ]]
diff --git a/src/lib/ecore_ipc/ecore_ipc.c b/src/lib/ecore_ipc/ecore_ipc.c
index 8cb7f3f7e8..0b27496d2f 100644
--- a/src/lib/ecore_ipc/ecore_ipc.c
+++ b/src/lib/ecore_ipc/ecore_ipc.c
@@ -481,7 +481,6 @@ ecore_ipc_server_add(Ecore_Ipc_Type type, const char *name, int port, const void
 
    if (efl_isa(svr->server, EFL_NET_SERVER_FD_CLASS))
      {
-        efl_net_server_fd_close_on_exec_set(svr->server, EINA_TRUE);
         efl_net_server_fd_reuse_address_set(svr->server, EINA_TRUE);
         efl_net_server_fd_reuse_port_set(svr->server, EINA_TRUE);
      }
@@ -495,7 +494,6 @@ ecore_ipc_server_add(Ecore_Ipc_Type type, const char *name, int port, const void
      {
         /* old ecore_con did not map ipv4 to ipv6... */
         efl_net_server_ssl_ipv6_only_set(svr->server, EINA_TRUE);
-        efl_net_server_ssl_close_on_exec_set(svr->server, EINA_TRUE);
         efl_net_server_ssl_reuse_address_set(svr->server, EINA_TRUE);
         efl_net_server_ssl_reuse_port_set(svr->server, EINA_TRUE);
      }