diff options
| author | Cedric BAIL <cedric@osg.samsung.com> | 2015-06-03 16:12:01 +0200 |
|---|---|---|
| committer | Cedric BAIL <cedric@osg.samsung.com> | 2015-06-03 17:14:12 +0200 |
| commit | 69e44c24e3bd946390585a123c73aac9a6653709 (patch) | |
| tree | 8d11db4a0057a8ef036163488625e72a37b8a7e5 | |
| parent | b2c47737b1f8cfa8762b7093b5b8d598c135fb73 (diff) | |
| download | efl-69e44c24e3bd946390585a123c73aac9a6653709.tar.gz | |
ecore_con: fix error handling case.
Since the move to Eo, in case of error we do not need to cleanup anything when
finalize finish. The kill function was arranged to do so, but it was still emiting
an event exposing the invalid Eo object. Looking closely at the code, that should
have not happened, as there was no symetrical ADD event and the function would do
nothing else. So removing it. This prevent any potential SEGV from a function that
would catch the DEL event.
@fix
| -rw-r--r-- | src/lib/ecore_con/ecore_con.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/lib/ecore_con/ecore_con.c b/src/lib/ecore_con/ecore_con.c index bc121d3965..941dc11cea 100644 --- a/src/lib/ecore_con/ecore_con.c +++ b/src/lib/ecore_con/ecore_con.c @@ -541,8 +541,6 @@ _ecore_con_connector_eo_base_finalize(Ecore_Con_Server *obj, void *pd EINA_UNUSE return obj; error: - if (svr->delete_me) return NULL; - _ecore_con_server_kill(obj); return NULL; } |