Fix coverity issue of Unchecked return value

Change-Id: I7ad59a9e75a5cbc7e1a802f914ff60cbfb483217

diff --git a/packaging/resource-manager.spec b/packaging/resource-manager.spec
index 293c130aee269ec4214d49115c5ec69f7005a22c..5d8229f011060c5733fc1b255d37374f35689f8a 100644 (file)
--- 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 fbf0c6e3ab6a60653a037d93401a53ee95163a56..b7513344e585b37390193fabbcc29cdf82b10506 100644 (file)
--- 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 7eb9f52c18887d57b25208a29814c264df12967f..66ba83f79987e9a24c293b8e9056c622545a97c1 100644 (file)
--- 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: