Fix ASAN issue

- The idle event could be called after handle is released
  if _recorder_create_common() is failed.

[Version] 0.3.23
[Issue Type] Bug fix

Change-Id: I25e6dd5fab51169eb6f92106717a7a60f9b640fb
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

diff --git a/packaging/capi-media-recorder.spec b/packaging/capi-media-recorder.spec
index 3afbbf6..31f0cff 100644 (file)
--- a/packaging/capi-media-recorder.spec
+++ b/packaging/capi-media-recorder.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-recorder
 Summary:    A Recorder API
-Version:    0.3.22
+Version:    0.3.23
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/src/recorder.c b/src/recorder.c
index e240071..d900a7b 100644 (file)
--- a/src/recorder.c
+++ b/src/recorder.c
@@ -1315,6 +1315,9 @@ static void _recorder_client_callback_destroy(recorder_cb_info_s *cb_info)
 
        g_return_if_fail(cb_info != NULL);
 
+       /* remove idle event */
+       _recorder_deactivate_idle_event_all(cb_info);
+
        LOGD("MSG receive thread[%p] destroy", cb_info->msg_recv_thread);
 
        g_thread_join(cb_info->msg_recv_thread);
@@ -1650,7 +1653,6 @@ int recorder_destroy(recorder_h recorder)
                LOGW("server disconnected. release resource without send message.");
 
        if (ret == RECORDER_ERROR_NONE) {
-               _recorder_deactivate_idle_event_all(pc->cb_info);
                _recorder_client_callback_destroy(pc->cb_info);
                g_free(pc);
                pc = NULL;