e_policy_visibility: modify log to avoid accessing freed resource 71/221371/1
authorDoyoun Kang <doyoun.kang@samsung.com>
Thu, 2 Jan 2020 01:49:20 +0000 (10:49 +0900)
committerDoyoun Kang <doyoun.kang@samsung.com>
Thu, 2 Jan 2020 02:12:16 +0000 (11:12 +0900)
Change-Id: I92afdffb83aa319c1eba97e1df90c7907ce57ee2

src/bin/e_policy_visibility.c

index 70d3e9eeb82582ebca105b99d587402c0078e7dc..9a9d744bd7755004397eb4fbcf528de999ec5742 100644 (file)
@@ -673,16 +673,14 @@ _e_vis_job_exec(Eina_Clist *elem)
    _e_vis_clist_unlink(elem);
    job = EINA_CLIST_ENTRY(elem, E_Vis_Job, entry);
 
+   VS_INF(job->vc->ec, "EXEC JOB:%p, type:%d (is_del ec:%d)", job, job->type, e_object_is_del(E_OBJECT(job->vc->ec)));
    /* After calling the function below, ec may have been deleted.
     * This is because the delayed ec deletion is perfromed in the following function.
     * Therefore, be careful when leaving out detailed log message about deleted ec
     * to avoid segmentation fault error.
     */
    _e_vis_client_job_exec(job->vc, job->type);
-   if (e_object_is_del(E_OBJECT(job->vc->ec)))
-     INF("VISIBILITY | FREE JOB:%p, type:%d | (ec:%p)", job, job->type, job->vc->ec);
-   else
-     VS_INF(job->vc->ec, "FREE JOB:%p, type:%d", job, job->type);
+   INF("VISIBILITY | FREE JOB:%p, type:%d", job, job->type);
    E_FREE_FUNC(job->timer, ecore_timer_del);
    free(job);
 }