ecore-con/legacy: kill server with DEL event if a dialer error occurs
authorMike Blumenkrantz <zmike@samsung.com>
Tue, 3 Jul 2018 16:42:00 +0000 (12:42 -0400)
committerBowon Ryu <bowon.ryu@samsung.com>
Wed, 4 Jul 2018 05:17:03 +0000 (14:17 +0900)
commit62f01907bf326df0d62e39a663d559af97316b11
tree63aabdb85bca505f131652dc3225a0b898375dc3
parentcc318a1b4b96905889d81bbec0d5691f0052a6de
ecore-con/legacy: kill server with DEL event if a dialer error occurs

Summary:
in the course of the efl-net rewrite, the previous relied-upon behavior
sequence of error -> disconnect -> del was broken, and error events no
longer triggered disconnects or server deletion.

the failure was caused by a check for the 'connecting' flag, preventing
DEL events from being triggered

further, the ordering of the events was flipped from ERROR -> DEL to
DEL -> ERROR; this is also fixed now

this caused any failed connection to leak the entire server connection
since existing handlers for the del event were never triggered and the
server was never destroyed

@fix
fix T6330

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6330

Differential Revision: https://phab.enlightenment.org/D6481
src/lib/ecore_con/ecore_con_legacy.c