[rtems commit] consolesimpletask.c: Fix Two Unchecked Return Values ( CID #1437625 and #1472765)

Joel Sherrill joel at rtems.org
Fri Feb 12 23:13:43 UTC 2021


Module:    rtems
Branch:    master
Commit:    14f5a0597a2030570fe05a7de3b5bd8254c31d75
Changeset: http://git.rtems.org/rtems/commit/?id=14f5a0597a2030570fe05a7de3b5bd8254c31d75

Author:    Ryan Long <ryan.long at oarcorp.com>
Date:      Fri Feb 12 15:38:23 2021 -0500

consolesimpletask.c: Fix Two Unchecked Return Values (CID #1437625 and #1472765)

CID 1437625: Unchecked return value from call to rtems_task_create().
CID 1472765: Unchecked return value from call to rtems_task_start().

Closes #4237.

---

 cpukit/libcsupport/src/consolesimpletask.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/cpukit/libcsupport/src/consolesimpletask.c b/cpukit/libcsupport/src/consolesimpletask.c
index 82ea2a5..393de3a 100644
--- a/cpukit/libcsupport/src/consolesimpletask.c
+++ b/cpukit/libcsupport/src/consolesimpletask.c
@@ -217,6 +217,7 @@ static const char _Console_simple_task_Name[] = "console";
 void _Console_simple_task_Initialize( void )
 {
   Console_simple_task_Control *cons;
+  rtems_status_code status;
 
   cons = &_Console_simple_task_Instance;
 
@@ -233,7 +234,7 @@ void _Console_simple_task_Initialize( void )
 
   IMFS_add_node( "/dev", &cons->Node, NULL );
 
-  rtems_task_create(
+  status = rtems_task_create(
     rtems_build_name('C', 'O', 'N', 'S'),
     RTEMS_MAXIMUM_PRIORITY - 1,
     RTEMS_MINIMUM_STACK_SIZE,
@@ -241,10 +242,12 @@ void _Console_simple_task_Initialize( void )
     RTEMS_DEFAULT_MODES,
     &cons->task
   );
+  _Assert_Unused_variable_equals(status, RTEMS_SUCCESSFUL);
 
-  rtems_task_start(
+  status = rtems_task_start(
     cons->task,
     _Console_simple_task_Task,
     (rtems_task_argument) cons
   );
+  _Assert_Unused_variable_equals(status, RTEMS_SUCCESSFUL);
 }



More information about the vc mailing list