From: YoungHun Kim <yh8004.kim@samsung.com>
Date: Fri, 13 Sep 2024 01:52:49 +0000 (+0900)
Subject: Fix coverity issue of Unchecked return value
X-Git-Tag: accepted/tizen/unified/20240913.150939^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fheads%2Faccepted%2Ftizen_unified_toolchain;p=platform%2Fcore%2Fmultimedia%2Fresource-manager.git

Fix coverity issue of Unchecked return value

Change-Id: I7ad59a9e75a5cbc7e1a802f914ff60cbfb483217
---

diff --git a/packaging/resource-manager.spec b/packaging/resource-manager.spec
index 293c130..5d8229f 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:    4
+Release:    5
 Group:      Multimedia/Libraries
 License:    Apache-2.0
 Source0:    %{name}-%{version}.tar.gz
diff --git a/src/rm_api.cpp b/src/rm_api.cpp
index fbf0c6e..b751334 100644
--- a/src/rm_api.cpp
+++ b/src/rm_api.cpp
@@ -106,7 +106,9 @@ int rm_register(rm_resource_cb cb, void *data, int *handle, rm_consumer_info *co
 	*handle = response.handle;
 
 	if (rm_init_cb(response.handle, cb, data) != RM_OK) {
-		rm_unregister(response.handle);
+		if (rm_unregister(response.handle) != RM_OK)
+			RM_ERR("failed to rm_unregister()");
+
 		RM_ERR("failed to init cb (%d)", response.handle);
 		rm_clear_cpu_inheritance();
 		return RM_ERROR;
diff --git a/src/rm_debug.cpp b/src/rm_debug.cpp
index 7eb9f52..66ba83f 100644
--- a/src/rm_debug.cpp
+++ b/src/rm_debug.cpp
@@ -22,7 +22,7 @@
 // LCOV_EXCL_START
 const char *rm_convert_requested_state_enum_to_string(rm_requests_resource_state_e state_enum)
 {
-	switch(state_enum) {
+	switch (state_enum) {
 		case RM_STATE_PASSIVE:
 			return "PASSIVE";
 		case RM_STATE_SHARABLE:
@@ -42,7 +42,7 @@ const char *rm_convert_requested_state_enum_to_string(rm_requests_resource_state
 
 const char *rm_convert_category_enum_to_string(rm_rsc_category_e category_enum)
 {
-	switch(category_enum) {
+	switch (category_enum) {
 		case RM_CATEGORY_NONE:
 			return "NONE";
 		case RM_CATEGORY_AUDIO_DECODER: