From de4dcb57315d5892b86e4c9f78358f8ed0a17ee4 Mon Sep 17 00:00:00 2001 From: Wentong Wu Date: Wed, 21 Aug 2019 20:30:24 +0800 Subject: [PATCH] tests: kernel: fix unhandled return values. Fix unhandled return values as most other places handled in this file, fix coverity issue 203454. Fixes: #18443. Signed-off-by: Wentong Wu --- tests/kernel/mem_protect/sys_sem/src/main.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/kernel/mem_protect/sys_sem/src/main.c b/tests/kernel/mem_protect/sys_sem/src/main.c index df5585b765b..07e2b90cb2d 100644 --- a/tests/kernel/mem_protect/sys_sem/src/main.c +++ b/tests/kernel/mem_protect/sys_sem/src/main.c @@ -116,8 +116,12 @@ void sem_take_multiple_high_prio_helper(void *p1, void *p2, void *p3) void sem_multiple_threads_wait_helper(void *p1, void *p2, void *p3) { + int ret_value; + /* get blocked until the test thread gives the semaphore */ - sys_sem_take(&multiple_thread_sem, K_FOREVER); + ret_value = sys_sem_take(&multiple_thread_sem, K_FOREVER); + zassert_true(ret_value == 0, + "sys_sem_take failed when its shouldn't have\n"); /* Inform the test thread that this thread has got multiple_thread_sem*/ sys_sem_give(&simple_sem);