commit e668b3fe7e108ba2ef44b6b69120470e72a165cb
author Cedric Auger <cauger@provenrun.com> Wed Sep 11 13:41:21 2019 +0200
committer Jérôme Forissier <jerome@forissier.org> Thu Sep 12 16:46:22 2019 +0200
tree 317ac8628ce88b5a1145f8a6b1733103e230b9e6
parent 110aac4733d1a7a1cf423bf2f73a42830141d7a3

ta: remove 0ms timeouts in the tested TAs

According to GP specification, a 0ms timeout is likely to trigger
a TEE_ERROR_CANCEL error rather than the expected TEE_SUCCESS.
TEE_ERROR_CANCEL is returned when the operation has been queued
by the TEE but did not yet reached the TA.

Signed-off-by: Cedric Auger <cedric.auger@provenrun.com>
Reviewed-by: Jerome Forissier <jerome@forissier.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>

ta/sdp_basic/ta_sdp_basic.c

diff --git a/ta/sdp_basic/ta_sdp_basic.c b/ta/sdp_basic/ta_sdp_basic.c
index aa2e65c..1decef5 100644
--- a/ta/sdp_basic/ta_sdp_basic.c
+++ b/ta/sdp_basic/ta_sdp_basic.c
@@ -274,7 +274,8 @@
         TEE_Result res = TEE_ERROR_GENERIC;
 
 	if (sess == TEE_HANDLE_NULL) {
-	        res = TEE_OpenTASession(&uuid, 0, 0, NULL, &sess, &ret_orig);
+	        res = TEE_OpenTASession(&uuid, TEE_TIMEOUT_INFINITE, 0, NULL,
+					&sess, &ret_orig);
 		if (res != TEE_SUCCESS) {
 	                EMSG("SDP basic test TA: TEE_OpenTASession() FAILED \n");
 	                goto cleanup_return;
@@ -282,7 +283,8 @@
 
         }
 
-        res = TEE_InvokeTACommand(sess, 0, nCommandID, nParamTypes, pParams, &ret_orig);
+        res = TEE_InvokeTACommand(sess, TEE_TIMEOUT_INFINITE,
+				  nCommandID, nParamTypes, pParams, &ret_orig);
         if (res != TEE_SUCCESS) {
                 EMSG("SDP basic test TA: TEE_OpenTASession() FAILED %x/%d\n",
 								res, ret_orig);
@@ -306,7 +308,8 @@
         TEE_Result res = TEE_ERROR_GENERIC;
 
 	if (sess == TEE_HANDLE_NULL) {
-	        res = TEE_OpenTASession(&uuid, 0, 0, NULL, &sess, &ret_orig);
+	        res = TEE_OpenTASession(&uuid, TEE_TIMEOUT_INFINITE, 0, NULL,
+					&sess, &ret_orig);
 		if (res != TEE_SUCCESS) {
 	                EMSG("SDP basic test TA: TEE_OpenTASession() FAILED \n");
 	                goto cleanup_return;
@@ -314,7 +317,8 @@
 
         }
 
-        res = TEE_InvokeTACommand(sess, 0, nCommandID, nParamTypes, pParams, &ret_orig);
+        res = TEE_InvokeTACommand(sess, TEE_TIMEOUT_INFINITE,
+				  nCommandID, nParamTypes, pParams, &ret_orig);
         if (res != TEE_SUCCESS) {
                 EMSG("SDP basic test TA: TEE_OpenTASession() FAILED %x/%d\n",
 								res, ret_orig);