From: Doyoun Kang <doyoun.kang@samsung.com>
Date: Thu, 2 Jan 2020 01:49:20 +0000 (+0900)
Subject: e_policy_visibility: modify log to avoid accessing freed resource
X-Git-Tag: submit/tizen/20200103.060921~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b7de61d48d9bf0e0b186c97cea79b0bfd828ef79;p=platform%2Fupstream%2Fenlightenment.git

e_policy_visibility: modify log to avoid accessing freed resource

Change-Id: I92afdffb83aa319c1eba97e1df90c7907ce57ee2
---

diff --git a/src/bin/e_policy_visibility.c b/src/bin/e_policy_visibility.c
index 70d3e9eeb8..9a9d744bd7 100644
--- a/src/bin/e_policy_visibility.c
+++ b/src/bin/e_policy_visibility.c
@@ -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);
 }