edje_program: add NULL check for ed->callbacks in edje_program
authorJEONGHYUN YUN <jh0506.yun@samsung.com>
Thu, 6 Apr 2017 21:26:35 +0000 (14:26 -0700)
committerCedric BAIL <cedric@osg.samsung.com>
Thu, 6 Apr 2017 22:20:49 +0000 (15:20 -0700)
Summary: Pointer ed->callbacks may have NULL value in callback add functions.

This reduce the chance of continue to kind of work for longer in case of memory
constrain. Maybe using Eina macro would be better.

Reviewers: jpeg, jypark

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4761

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/lib/edje/edje_program.c

index 25f8179..2889ed1 100644 (file)
@@ -239,6 +239,7 @@ edje_object_propagate_callback_add(Evas_Object *obj, void (*func)(void *data, Ev
 
    if (!ed->callbacks)
      ed->callbacks = _edje_signal_callback_alloc();
+   if (!ed->callbacks) return;
 
    sig = eina_stringshare_add("*");
    src = eina_stringshare_add("*");
@@ -266,6 +267,7 @@ _edje_object_signal_callback_add(Eo *obj EINA_UNUSED, Edje *ed, const char *emis
 
    if (!ed->callbacks)
      ed->callbacks = _edje_signal_callback_alloc();
+   if (!ed->callbacks) return;
 
    _edje_signal_callback_push(ed->callbacks,
                               emission, source,