rggammon
/
efl
forked from enlightenment/efl
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix leak in elm atspi 

First..
Revert "elementary: Remove defined but not used function"
This reverts commit cfc167859a.

This func should exist. it has a reason to exist.. it somehow was not
being called anymore. this fixes that and calls it to avoid a leak.

@fix
This commit is contained in:
Carsten Haitzler 2019-03-05 17:05:03 +00:00
parent cfc167859a
commit 8f9cbdc09e
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/lib/elementary/elm_atspi_bridge.c
View File

@ -4597,6 +4597,19 @@ _key_event_info_new(int event_type, const Ecore_Event_Key *data, Eo *bridge)
return ret;
}
static void
_key_event_info_free(Key_Event_Info *data)
{
EINA_SAFETY_ON_NULL_RETURN(data);
eina_stringshare_del(data->event.keyname);
eina_stringshare_del(data->event.key);
eina_stringshare_del(data->event.string);
eina_stringshare_del(data->event.compose);
free(data);
}
static short
_ecore_modifiers_2_atspi(unsigned int modifiers)
{
@ -4679,6 +4692,7 @@ _elm_atspi_bridge_key_filter(void *data, void *loop EINA_UNUSED, int type, void
iter = eldbus_message_iter_get(req);
_iter_marshall_key_event(iter, ke);
_key_event_info_free(ke);
// timeout should be kept reasonably low to avoid delays
if (!(reply = eldbus_proxy_send_and_block(proxy, req, 100)))