tests: check return value

Change-Id: I15bc563500889e26f42600a6066dd0fe884fc29f

diff --git a/tools/tdm_test_client.c b/tools/tdm_test_client.c
index a8e7c62a47a07d84046ea9e3a4f8fe25d4ebb32f..11976fbcfd2bf3af7fbbd716508c9728fd14dce9 100644 (file)
--- 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 bf5a8e2f3c1e32a3dffb89557e632f5a3075a105..17c0121a599f4227f8a425592a6928c722bfee8b 100644 (file)
--- 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) {