michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: posix: pthread: correct cases for pthread_timedjoin_np()

Browse source 
pthread_timedjoin_np() is allowed to take a negative tv_sec
parameter, since that is still a valid timespec.

Signed-off-by: Chris Friedt <cfriedt@tenstorrent.com>
This commit is contained in:
Chris Friedt 2025-04-23 08:44:27 -04:00 • committed by Benjamin Cabé
commit 3d146aed05
1 changed files with 3 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
tests/posix/common/src/pthread.c
View file

@ -404,9 +404,8 @@ ZTEST(pthread, test_pthread_timedjoin)
struct timespec not_done;
struct timespec done;
struct timespec invalid[] = {
[0] = {.tv_sec = -1},
[1] = {.tv_nsec = -1},
[2] = {.tv_nsec = NSEC_PER_SEC},
{.tv_nsec = -1},
{.tv_nsec = NSEC_PER_SEC},
};
/* setup timespecs when the thread is still running and when it is done */
@ -426,7 +425,7 @@ ZTEST(pthread, test_pthread_timedjoin)
/* Creating a thread that exits after 200ms*/
zassert_ok(pthread_create(&th, NULL, timedjoin_thread, INT_TO_POINTER(sleep_duration_ms)));
/* pthread_timedjoin-np must return -EINVAL for invalid struct timespecs */
/* pthread_timedjoin-np must return EINVAL for invalid struct timespecs */
zassert_equal(pthread_timedjoin_np(th, &ret, NULL), EINVAL);
for (size_t i = 0; i < ARRAY_SIZE(invalid); ++i) {
zassert_equal(pthread_timedjoin_np(th, &ret, &invalid[i]), EINVAL);