From: Seungbae Shin <seungbae.shin@samsung.com>
Date: Wed, 28 Aug 2019 07:35:43 +0000 (+0900)
Subject: testsuite : check return value of API properly.
X-Git-Tag: submit/tizen/20190903.063631~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F28%2F212928%2F1;p=platform%2Fcore%2Fapi%2Fsound-manager.git

testsuite : check return value of API properly.

[Version] 0.5.39
[Issue Type] Coverity

Change-Id: I0a9450ca656b64d579591f367be74d15d2a6caf7
---

diff --git a/packaging/capi-media-sound-manager.spec b/packaging/capi-media-sound-manager.spec
index b310786..c200520 100644
--- a/packaging/capi-media-sound-manager.spec
+++ b/packaging/capi-media-sound-manager.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-sound-manager
 Summary:    Sound Manager library
-Version:    0.5.38
+Version:    0.5.39
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/test/sound_manager_test.c b/test/sound_manager_test.c
index 1309950..8e2f266 100644
--- a/test/sound_manager_test.c
+++ b/test/sound_manager_test.c
@@ -1770,20 +1770,25 @@ static void interpret(char *cmd)
 				break;
 			}
 
-			sound_manager_get_device_list(SOUND_DEVICE_TYPE_INTERNAL_MASK, &device_list);
-			while (!sound_manager_get_next_device(device_list, &device)) {
-				if (!sound_manager_get_device_id(device, &id)) {
-					if (in_device_id && id == in_device_id) {
-						sound_manager_get_device_type(device, &type);
-						g_print("preferred [ IN] device type[%s], id[%d]\n", g_device_type_str[type], in_device_id);
-					}
-					if (out_device_id && id == out_device_id) {
-						sound_manager_get_device_type(device, &type);
-						g_print("preferred [OUT] device type[%s], id[%d]\n", g_device_type_str[type], out_device_id);
+			ret = sound_manager_get_device_list(SOUND_DEVICE_TYPE_INTERNAL_MASK, &device_list);
+			if (ret) {
+				g_print("failed to get device list(), ret(0x%x)\n", ret);
+			} else {
+				while (!sound_manager_get_next_device(device_list, &device)) {
+					if (!sound_manager_get_device_id(device, &id)) {
+						if (in_device_id && id == in_device_id) {
+							if (sound_manager_get_device_type(device, &type) == SOUND_MANAGER_ERROR_NONE)
+								g_print("preferred [ IN] device type[%s], id[%d]\n", g_device_type_str[type], in_device_id);
+						}
+						if (out_device_id && id == out_device_id) {
+							if (sound_manager_get_device_type(device, &type) == SOUND_MANAGER_ERROR_NONE)
+								g_print("preferred [OUT] device type[%s], id[%d]\n", g_device_type_str[type], out_device_id);
+						}
 					}
 				}
+
+				sound_manager_free_device_list(device_list);
 			}
-			sound_manager_free_device_list(device_list);
 		}
 		reset_menu_state();
 		break;