This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git

commit e0e9f3d7e564d3c746a1d9409869d46c0bafb88f
Author: ouyangxiangzhen <[email protected]>
AuthorDate: Tue Dec 31 09:45:44 2024 +0800

    apps/testing: Fix Coverity.
    
    This commit is to make Coverity happy.
    
    Signed-off-by: ouyangxiangzhen <[email protected]>
---
 testing/sched/timerjitter/timerjitter.c | 29 +++++++++++++++++++++++++----
 1 file changed, 25 insertions(+), 4 deletions(-)

diff --git a/testing/sched/timerjitter/timerjitter.c 
b/testing/sched/timerjitter/timerjitter.c
index 214bbf735..3a73ee298 100644
--- a/testing/sched/timerjitter/timerjitter.c
+++ b/testing/sched/timerjitter/timerjitter.c
@@ -137,12 +137,15 @@ static FAR void *timerjitter(FAR void *arg)
   struct timespec   next;
   struct timespec   intv;
   struct itimerspec tspec;
-  struct sigevent   sigev;
   sigset_t          sigset;
   timer_t           timer;
   int64_t           diff;
   int               sigs;
   int               ret;
+  struct sigevent   sigev =
+  {
+    0
+  };
 
   sigemptyset(&sigset);
   sigaddset(&sigset, SIGALRM);
@@ -154,8 +157,20 @@ static FAR void *timerjitter(FAR void *arg)
   sigev.sigev_notify = SIGEV_SIGNAL;
   sigev.sigev_signo  = SIGALRM;
 
-  timer_create(param->clockid, &sigev, &timer);
-  clock_gettime(param->clockid, &now);
+  ret = timer_create(param->clockid, &sigev, &timer);
+
+  if (ret != 0)
+    {
+      printf("timer_create failed %d\n", ret);
+      return NULL;
+    }
+
+  ret = clock_gettime(param->clockid, &now);
+
+  if (ret)
+    {
+      printf("clock_gettime failed %d\n", ret);
+    }
 
   next = now;
   calc_next(&next, &intv);
@@ -167,7 +182,13 @@ static FAR void *timerjitter(FAR void *arg)
   /* Using TIMER_ABSTIME */
 
   tspec.it_value = next;
-  timer_settime(timer, TIMER_ABSTIME, &tspec, NULL);
+  ret = timer_settime(timer, TIMER_ABSTIME, &tspec, NULL);
+
+  if (ret)
+    {
+      printf("timer_settime failed %d\n", ret);
+      return NULL;
+    }
 
   param->avg = 0;
   param->max = 0;

Reply via email to