ecore_ipc: remove efl_loop_fd_set before finishing the unix sockect connection 99/226799/2
authorYeongjong Lee <yj34.lee@samsung.com>
Thu, 5 Mar 2020 10:03:17 +0000 (19:03 +0900)
committerJongmin Lee <jm105.lee@samsung.com>
Mon, 9 Mar 2020 02:52:35 +0000 (02:52 +0000)
For Efl.Net.Dialer_Unix, fd should be set when the socket is finished connecting.

efl_loop_fd_set is called in the `_efl_net_dialer_unix_connected` which is
called at the end of connection thread named `_efl_net_connect_async_end`.
That means we don't need to set fd right after `efl_add(EFL_NET_DIALER_UNIX_CLASS`.

Also, this will fix intermittent efl-test-suite failure related to ecore_evas_extn.

@tizen_fix

Change-Id: I2f884e3de49be109cc945c76f42d22c206e10c1b

src/lib/ecore_ipc/ecore_ipc.c

index 609ee9d33c8dabab268a99811c6d0797c43d0c7d..f756693a9d9d771fa36a73fdada00e2085717d8b 100644 (file)
@@ -713,7 +713,6 @@ ecore_ipc_server_with_fd_connect(Ecore_Ipc_Type type, char *name, int port, int
 
         svr->dialer.dialer = efl_add(EFL_NET_DIALER_UNIX_CLASS, efl_main_loop_get());
         EINA_SAFETY_ON_NULL_GOTO(svr->dialer.dialer, error_dialer);
-        efl_loop_fd_set(svr->dialer.dialer, fd);// TIZEN_ONLY(20180125): smack issue
      }
    else if ((type & ECORE_IPC_TYPE) == ECORE_IPC_LOCAL_SYSTEM)
      {
@@ -722,7 +721,6 @@ ecore_ipc_server_with_fd_connect(Ecore_Ipc_Type type, char *name, int port, int
 
         svr->dialer.dialer = efl_add(EFL_NET_DIALER_UNIX_CLASS, efl_main_loop_get());
         EINA_SAFETY_ON_NULL_GOTO(svr->dialer.dialer, error_dialer);
-        efl_loop_fd_set(svr->dialer.dialer, fd);// TIZEN_ONLY(20180125): smack issue
      }
 #endif /* EFL_NET_DIALER_UNIX_CLASS */
 #ifdef EFL_NET_DIALER_WINDOWS_CLASS