e_policy_visibility: add null checking code in timeout functions 23/275923/1 accepted/tizen/6.5/unified/20220607.134940 submit/tizen_6.5/20220607.025827
authorDoyoun Kang <doyoun.kang@samsung.com>
Thu, 2 Jun 2022 23:59:04 +0000 (08:59 +0900)
committerDoyoun Kang <doyoun.kang@samsung.com>
Mon, 6 Jun 2022 22:22:05 +0000 (07:22 +0900)
Change-Id: If2dac2e6b6d63f4060bc225af888c682c2c1f5a4

src/bin/e_policy_visibility.c

index 4eeb6ee1864c679982775f084d8ccba3bcad40c4..4fdcd2aedebcb4cb5cdf1a9d9270dbe93226ccd6 100644 (file)
@@ -987,7 +987,10 @@ static Eina_Bool
 _e_vis_client_grab_cb_timeout(void *data)
 {
    E_Vis_Grab *grab = data;
-   VS_INF(grab->vc->ec, "TIMEOUT(%f) Grab %s", e_config->deiconify_pending_timeout, grab->name);
+   if (!grab) return ECORE_CALLBACK_DONE;
+
+   ELOGF("POL_VIS", "TIMEOUT(%f) Grab(%p), name: %s", NULL, e_config->deiconify_pending_timeout, grab, grab->name);
+   ELOGF("POL_VIS", "Info is_del:%d", grab->vc->ec, e_object_is_del(E_OBJECT(grab->vc->ec)));
    grab->timer = NULL;
    grab->deleted = 1;
    _e_vis_client_grab_remove(grab->vc, grab);
@@ -1000,12 +1003,12 @@ _e_vis_client_grab_get(E_Vis_Client *vc, E_Vis_Job_Type type, const char *name)
 {
    E_Vis_Grab *grab;
 
-   VS_INF(vc->ec, "Get job grab: '%s'", name);
-
    grab = _e_vis_grab_new(vc, type, name, _e_vis_client_grab_cb_timeout);
    if (!grab)
      return NULL;
 
+   VS_INF(vc->ec, "Get job Grab(%p), name: %s", grab, name);
+
    _e_vis_client_grab_add(vc, grab);
 
    return grab;
@@ -1157,8 +1160,10 @@ static Eina_Bool
 _e_vis_client_job_timeout(void *data)
 {
    E_Vis_Job *job = data;
+   if (!job) return ECORE_CALLBACK_DONE;
 
-   VS_INF(job->vc->ec, "TIMEOUT(%f) JOB %d", e_config->deiconify_pending_timeout, job->type);
+   ELOGF("POL_VIS", "TIMEOUT(%f) JOB:%p, type:%d", NULL, e_config->deiconify_pending_timeout, job, job->type);
+   ELOGF("POL_VIS", "Info is_del:%d", job->vc->ec, e_object_is_del(E_OBJECT(job->vc->ec)));
    job->timer = NULL;
 
    /* FIXME delete all grab and evaluate it instead of exec */