From: Boram Park <boram1288.park@samsung.com>
Date: Mon, 19 Mar 2018 06:58:12 +0000 (+0900)
Subject: tests: check return value
X-Git-Tag: submit/tizen/20180320.093933~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d37f0aa105da59dbb0601f4dbf47cbc93ae7fd0c;p=platform%2Fcore%2Fuifw%2Flibtdm.git

tests: check return value

Change-Id: I15bc563500889e26f42600a6066dd0fe884fc29f
---

diff --git a/tools/tdm_test_client.c b/tools/tdm_test_client.c
index a8e7c62a..11976fbc 100644
--- a/tools/tdm_test_client.c
+++ b/tools/tdm_test_client.c
@@ -273,9 +273,12 @@ do_query(tdm_test_client *data)
 		return;
 	}
 
-	tdm_client_output_get_conn_status(output, &status);
-	tdm_client_output_get_dpms(output, &dpms);
-	tdm_client_output_get_refresh_rate(output, &refresh);
+	error = tdm_client_output_get_conn_status(output, &status);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	error = tdm_client_output_get_dpms(output, &dpms);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	error = tdm_client_output_get_refresh_rate(output, &refresh);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
 
 	printf("tdm_output \"%s\"\n", data->args.output_name);
 	printf("\tstatus : %s\n", conn_str[status]);
@@ -298,7 +301,8 @@ do_vblank(tdm_test_client *data)
 		return;
 	}
 
-	tdm_client_output_add_change_handler(output, _client_output_handler, NULL);
+	error = tdm_client_output_add_change_handler(output, _client_output_handler, NULL);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
 
 	vblank = tdm_client_output_create_vblank(output, &error);
 	if (error != TDM_ERROR_NONE) {
@@ -306,12 +310,18 @@ do_vblank(tdm_test_client *data)
 		return;
 	}
 
-	tdm_client_vblank_set_name(vblank, data->args.vblank_name);
-	tdm_client_vblank_set_enable_fake(vblank, data->args.enable_fake);
-	tdm_client_vblank_set_sync(vblank, data->args.sync);
-	if (data->args.fps > 0)
-		tdm_client_vblank_set_fps(vblank, data->args.fps);
-	tdm_client_vblank_set_offset(vblank, data->args.offset);
+	error = tdm_client_vblank_set_name(vblank, data->args.vblank_name);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	error = tdm_client_vblank_set_enable_fake(vblank, data->args.enable_fake);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	error = tdm_client_vblank_set_sync(vblank, data->args.sync);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	if (data->args.fps > 0) {
+		error = tdm_client_vblank_set_fps(vblank, data->args.fps);
+		TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
+	}
+	error = tdm_client_vblank_set_offset(vblank, data->args.offset);
+	TDM_WARNING_IF_FAIL(error == TDM_ERROR_NONE);
 
 	error = tdm_client_get_fd(data->client, &fd);
 	if (error != TDM_ERROR_NONE || fd < 0) {
diff --git a/tools/tdm_test_server.c b/tools/tdm_test_server.c
index bf5a8e2f..17c0121a 100644
--- a/tools/tdm_test_server.c
+++ b/tools/tdm_test_server.c
@@ -819,7 +819,8 @@ destroy(tdm_test_server *data)
 			ret = tdm_layer_unset_buffer(l->layer);
 			TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 		}
-		tdm_layer_commit(l->layer, NULL, NULL);
+		ret = tdm_layer_commit(l->layer, NULL, NULL);
+		TDM_EXIT_IF_FAIL(ret == TDM_ERROR_NONE);
 	}
 
 	LIST_FOR_EACH_ENTRY_SAFE(p, pp, &data->pp_list, link) {