[EGIT] [core/efl] master 01/01: exactness: check action for NULL

[Stefan Schmidt]

stefan pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f5516e896d690a233c1747cdc7c893db15c240c1

commit f5516e896d690a233c1747cdc7c893db15c240c1
Author: Stefan Schmidt <s.schm...@samsung.com>
Date:   Wed Apr 22 17:44:04 2020 +0200

    exactness: check action for NULL
    
    If the given events list is NULL the data pointer would be as well. Make
    sure we check for NULL here before accessing.
    
    CID: 1419843
    
    Reviewed-by: Marcel Hollerbach <m...@marcel-hollerbach.de>
    Differential Revision: https://phab.enlightenment.org/D11754
---
 src/bin/exactness/player.c | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/bin/exactness/player.c b/src/bin/exactness/player.c
index 1e759e4c9b..befd028852 100644
--- a/src/bin/exactness/player.c
+++ b/src/bin/exactness/player.c
@@ -508,11 +508,14 @@ _feed_event_timer_cb(void *data EINA_UNUSED)
      }
    else
      {
-        if (act->type != EXACTNESS_ACTION_STABILIZE)
+        if (act && act->type != EXACTNESS_ACTION_STABILIZE)
           {
              act = eina_list_data_get(_cur_event_list);
-             DBG("  %s timer_time=<%f>\n", __func__, act->delay_ms / 1000.0);
-             ecore_timer_add(act->delay_ms / 1000.0, _feed_event_timer_cb, 
NULL);
+             if (act && act->delay_ms)
+               {
+                  DBG("  %s timer_time=<%f>\n", __func__, act->delay_ms / 
1000.0);
+                  ecore_timer_add(act->delay_ms / 1000.0, 
_feed_event_timer_cb, NULL);
+               }
           }
      }
    return ECORE_CALLBACK_CANCEL;
@@ -549,8 +552,11 @@ _stabilization_timer_cb(void *data EINA_UNUSED)
         if (_src_type != FTYPE_REMOTE && !_pause_request)
           {
              Exactness_Action *act = eina_list_data_get(_cur_event_list);
-             DBG("  %s timer_time=<%f>\n", __func__, act->delay_ms / 1000.0);
-             ecore_timer_add(act->delay_ms / 1000.0, _feed_event_timer_cb, 
NULL);
+             if (act && act->delay_ms)
+               {
+                  DBG("  %s timer_time=<%f>\n", __func__, act->delay_ms / 
1000.0);
+                  ecore_timer_add(act->delay_ms / 1000.0, 
_feed_event_timer_cb, NULL);
+               }
           }
         need_more = STAB_MAX;
         return ECORE_CALLBACK_CANCEL;

--