regression 1008: test open session failure once, unless level > 0
Test 1008 runs a negative test for TEEC_OpenSession() 100 times in a
loop, which can be useful to detect and troubleshoot memory leaks.
However the loop increases the execution time and the amount of logs
produced, so it is better reserved for level > 0 (xtest -l level).
Note, xtest 1013 reduces the number of loops when level == 0 already.
Signed-off-by: Jerome Forissier <jerome@forissier.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
diff --git a/host/xtest/regression_1000.c b/host/xtest/regression_1000.c
index a672a81..2bb7ae6 100644
--- a/host/xtest/regression_1000.c
+++ b/host/xtest/regression_1000.c
@@ -749,19 +749,15 @@
{
size_t n = 0;
- (void)ADBG_EXPECT_TEEC_RESULT(c, TEEC_ERROR_GENERIC,
- xtest_teec_open_session(&session_crypt,
- &create_fail_test_ta_uuid, NULL,
- &ret_orig));
- /*
- * Run this several times to see that there's no memory leakage.
- */
for (n = 0; n < 100; n++) {
Do_ADBG_Log("n = %zu", n);
(void)ADBG_EXPECT_TEEC_RESULT(c, TEEC_ERROR_GENERIC,
xtest_teec_open_session(&session_crypt,
&create_fail_test_ta_uuid,
NULL, &ret_orig));
+ /* level > 0 may be used to detect/debug memory leaks */
+ if (!level)
+ break;
}
}
Do_ADBG_EndSubCase(c, "Create session fail");