summaryrefslogtreecommitdiff
path: root/src/modules/emotion/generic
diff options
context:
space:
mode:
authorMike Blumenkrantz <zmike@samsung.com>2018-06-19 13:29:16 -0400
committerMike Blumenkrantz <zmike@samsung.com>2018-06-19 13:29:16 -0400
commit90cb995ac9f5fce96f5a95dfdd2598d5d4cdb726 (patch)
tree5c768d3018efa54ed0a897496eb9afef42d538cd /src/modules/emotion/generic
parentba771f8b698f9239c82edde45db85e92d81b50a4 (diff)
emotion_generic: clean up event handlers on fork failure
Summary: this case would guarantee a crash if the module was ever unloaded and the event handlers were called, and it leaks otherwise @fix Reviewers: ManMower, devilhorns, bu5hm4n Reviewed By: ManMower, bu5hm4n Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6296
Diffstat (limited to 'src/modules/emotion/generic')
-rw-r--r--src/modules/emotion/generic/emotion_generic.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/modules/emotion/generic/emotion_generic.c b/src/modules/emotion/generic/emotion_generic.c
index 4aacecad4b..601e9e2d92 100644
--- a/src/modules/emotion/generic/emotion_generic.c
+++ b/src/modules/emotion/generic/emotion_generic.c
@@ -933,6 +933,10 @@ _fork_and_exec(Emotion_Generic_Video *ev)
933 933
934 if (!_player_exec(ev)) 934 if (!_player_exec(ev))
935 { 935 {
936 eina_stringshare_del(ev->shmname);
937 ecore_event_handler_del(ev->player_add);
938 ecore_event_handler_del(ev->player_del);
939 ecore_event_handler_del(ev->player_data);
936 ERR("could not start player."); 940 ERR("could not start player.");
937 return EINA_FALSE; 941 return EINA_FALSE;
938 } 942 }