Remove g_thread_unref after g_thread_join

g_thread_unref is also called in g_thread_join,
so g_thread_unref after g_thread_join is not needed
and it can cause memory corruption.

[Version] 0.2.49
[Profile] Common
[Issue Type] Bug fix
[Dependency module] N/A
[Test] [M(T) - Boot=(OK), sdb=(OK), Home=(OK), Touch=(OK), Version=tizen-3.0-mobile_20161230.2]

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

diff --git a/packaging/capi-media-recorder.spec b/packaging/capi-media-recorder.spec
index 0f5d4f0..8816d29 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.2.48
+Version:    0.2.49
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/src/recorder.c b/src/recorder.c
index f0116e6..b799ef1 100644 (file)
--- a/src/recorder.c
+++ b/src/recorder.c
@@ -1090,7 +1090,6 @@ static void __destroy_msg_handler_thread(recorder_msg_handler_info_s *handler_in
        g_mutex_unlock(&handler_info->mutex);
 
        g_thread_join(handler_info->thread);
-       g_thread_unref(handler_info->thread);
        handler_info->thread = NULL;
 
        g_mutex_clear(&handler_info->mutex);
@@ -1330,7 +1329,6 @@ static void _recorder_client_callback_destroy(recorder_cb_info_s *cb_info)
        LOGD("MSG receive thread[%p] destroy", cb_info->msg_recv_thread);
 
        g_thread_join(cb_info->msg_recv_thread);
-       g_thread_unref(cb_info->msg_recv_thread);
        cb_info->msg_recv_thread = NULL;
 
        LOGD("msg_recv thread removed");