[0.3.139] Fix ASAN bug(heap-use-after-free)

Change-Id: I3e10f6a9f82b3ae4e89733ba136dcd45b58f56cb

diff --git a/packaging/capi-media-player.spec b/packaging/capi-media-player.spec
index f384b87..92c7b6e 100644 (file)
--- a/packaging/capi-media-player.spec
+++ b/packaging/capi-media-player.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-player
 Summary:    A Media Player API
-Version:    0.3.138
+Version:    0.3.139
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/src/player.c b/src/player.c
index 4487dea..edcd82b 100644 (file)
--- a/src/player.c
+++ b/src/player.c
@@ -1522,10 +1522,10 @@ static void _player_remove_idle_event(callback_cb_info_s *cb_info, muse_player_e
                                LOGD("remove idle event [%p:%d]", event_data, event_data->int_data);
                                source = g_main_context_find_source_by_user_data (
                                                        g_main_context_default (), event_data);
-                               ret = g_idle_remove_by_data(event_data);
                                if (source)
                                        check_in_call = source->flags & G_HOOK_FLAG_IN_CALL;
 
+                               ret = g_idle_remove_by_data(event_data);
                                if (!ret || check_in_call) {
                                        /* will be handled at _player_event_job_function() as an exception */
                                        event_data->cb_info = NULL;