Skip to content

Commit

Permalink
[review] xtest: regression: add case 1040
Browse files Browse the repository at this point in the history
Addressing comments.

Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
  • Loading branch information
jenswi-linaro committed Sep 5, 2023
1 parent f135dd7 commit ac2cd38
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions host/xtest/regression_1000.c
Original file line number Diff line number Diff line change
Expand Up @@ -3305,11 +3305,15 @@ static void *test_1040_thread(void *arg)
&err_orig);
TEEC_CloseSession(&session);
if (res != TEEC_ERROR_TARGET_DEAD) {
a->res = res;
if (res)
a->res = res;
else
a->res = TEEC_ERROR_GENERIC;
return NULL;
}
n++;
}
a->res = TEEC_SUCCESS;
return NULL;
}

Expand All @@ -3323,7 +3327,7 @@ static void xtest_tee_test_1040(ADBG_Case_t *c)
for (n = 0; n < nt; n++) {
if (!ADBG_EXPECT(c, 0, pthread_create(&arg[n].thr, NULL,
test_1040_thread,
arg + n)))
arg + n)))
nt = n; /* break loop and start cleanup */
}
for (n = 0; n < nt; n++) {
Expand All @@ -3335,4 +3339,3 @@ static void xtest_tee_test_1040(ADBG_Case_t *c)
}
ADBG_CASE_DEFINE(regression, 1040, xtest_tee_test_1040,
"Test panic in concurrent open/invoke/close session");

0 comments on commit ac2cd38

Please sign in to comment.