video: Fix use after free

To avoid null dereference, it deletes post_client_idler_before_hook at
the beginning of the callback function.

Change-Id: I90f627752ae39fa26ffbcea1a0daf7b4b5f38639

diff --git a/src/bin/displaymgr/video/e_video_hwc.c b/src/bin/displaymgr/video/e_video_hwc.c
index 2d9d0f7ddf43052e01f04219ee6ac2268acb36b9..f5eb61be87fbd1e14f92a84ef6bbb32895921b91 100644 (file)
--- a/src/bin/displaymgr/video/e_video_hwc.c
+++ b/src/bin/displaymgr/video/e_video_hwc.c
@@ -1230,6 +1230,8 @@ _e_video_hwc_cb_post_client_idler_before(void *data)
 
    evh = data;
 
+   E_FREE_FUNC(evh->render.post_client_idler_before_hook, e_main_hook_del);
+
    if (evh->render.map)
      {
         evh->render.map = EINA_FALSE;
@@ -1263,8 +1265,6 @@ _e_video_hwc_cb_post_client_idler_before(void *data)
              evh->backend.destroy(evh);
           }
      }
-
-   E_FREE_FUNC(evh->render.post_client_idler_before_hook, e_main_hook_del);
 }
 
 static void