From: YoungHun Kim <yh8004.kim@samsung.com>
Date: Tue, 27 Aug 2024 08:23:43 +0000 (+0900)
Subject: fixup! Fix the svace and coverity issue
X-Git-Tag: accepted/tizen/unified/20240828.164019^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f85112d393c9b055176071a5e921b2311f54362b;p=platform%2Fcore%2Fmultimedia%2Fresource-manager.git

fixup! Fix the svace and coverity issue

Change-Id: Id5e9d965e554a85c5efc422e997c43df7eb934dd
---

diff --git a/packaging/resource-manager.spec b/packaging/resource-manager.spec
index ec83557..6e74131 100644
--- a/packaging/resource-manager.spec
+++ b/packaging/resource-manager.spec
@@ -1,7 +1,7 @@
 Name:       resource-manager
 Summary:    Resource manager
 Version:    0.1
-Release:    2
+Release:    3
 Group:      Multimedia/Libraries
 License:    Apache-2.0
 Source0:    %{name}-%{version}.tar.gz
diff --git a/src/rm_callback.cpp b/src/rm_callback.cpp
index 75a2cf5..385d130 100644
--- a/src/rm_callback.cpp
+++ b/src/rm_callback.cpp
@@ -61,6 +61,7 @@ static pthread_mutex_t rm_cb_mutex = PTHREAD_MUTEX_INITIALIZER;
 
 static GMainLoop *rm_client_loop = NULL;
 static GMainContext *rm_client_context = NULL;
+static GThread *rm_thrd = NULL;
 
 static void rm_add_main_loop(GMainLoop *loop)
 {
@@ -86,6 +87,8 @@ static gpointer rm_run_loop(gpointer data)
 	if (main_loop != NULL)
 		g_main_loop_run(main_loop);
 
+	g_thread_join(rm_thrd);
+
 	RM_INFO("RM client thread finished");
 	return NULL;
 }
@@ -100,7 +103,7 @@ int _create_event_loop(void)
 	GMainLoop *main_loop = g_main_loop_new(rm_client_ctx, FALSE);
 	rm_add_main_loop(main_loop);
 
-	GThread *rm_thrd = g_thread_new("rm_client", rm_run_loop, NULL);
+	rm_thrd = g_thread_new("rm_client", rm_run_loop, NULL);
 
 	if (rm_thrd == NULL) {
 		// LCOV_EXCL_START
diff --git a/ut/testcase/ut_main.cpp b/ut/testcase/ut_main.cpp
index 8a5d5d5..23876a3 100644
--- a/ut/testcase/ut_main.cpp
+++ b/ut/testcase/ut_main.cpp
@@ -45,7 +45,7 @@ GTEST_API_ int main(int argc, char **argv)
 
 	//GMainContext *context = g_main_context_new();
 	g_main_loop = g_main_loop_new(NULL, FALSE);
-	GThread *cb_thread = g_thread_new("rc_cb_thread", tc_run_loop, g_main_loop);
+	g_autoptr(GThread) cb_thread = g_thread_new("rc_cb_thread", tc_run_loop, g_main_loop);
 
 	if (cb_thread == NULL) {
 		g_main_loop_unref(g_main_loop);