e_policy_visibility: fixed a heap-use-after-free error for the deleted ec when logging

Change-Id: Iae4f57ba35332aa532d187320d93c5e00aa7f4d1

diff --git a/src/bin/e_policy_visibility.c b/src/bin/e_policy_visibility.c
index 4711bbf..a967761 100644 (file)
--- a/src/bin/e_policy_visibility.c
+++ b/src/bin/e_policy_visibility.c
@@ -653,8 +653,17 @@ _e_vis_job_exec(Eina_Clist *elem)
 
    _e_vis_clist_unlink(elem);
    job = EINA_CLIST_ENTRY(elem, E_Vis_Job, entry);
+
+   /* 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);
-   VS_INF(job->vc->ec, "FREE JOB:%p, type:%d", job, 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);
    E_FREE_FUNC(job->timer, ecore_timer_del);
    free(job);
 }