2009-09-28 Joel Sherrill <joel.sherrill@OARcorp.com>

* sp09/screen01.c, sp09/screen02.c, sp09/screen03.c, sp09/screen05.c, sp09/screen09.c, sp09/screen14.c, sp09/sp09.scn, sp28/init.c, sp28/sp28.scn, sp43/init.c: Add more error cases uncovered by uC5282 coverage runs.
2025-12-26 14:18:20 +00:00 · 2009-09-28 23:07:56 +00:00
parent 6fd4d06f39
commit f55f60591c
11 changed files with 244 additions and 13 deletions
--- a/testsuites/sptests/ChangeLog
+++ b/testsuites/sptests/ChangeLog
@@ -1,3 +1,10 @@
+2009-09-28	Joel Sherrill <joel.sherrill@OARcorp.com>
+
+	* sp09/screen01.c, sp09/screen02.c, sp09/screen03.c, sp09/screen05.c,
+	sp09/screen09.c, sp09/screen14.c, sp09/sp09.scn, sp28/init.c,
+	sp28/sp28.scn, sp43/init.c: Add more error cases uncovered by uC5282
+	coverage runs.
+
 2009-09-28	Joel Sherrill <joel.sherrill@OARcorp.com>

 	* sp09/screen02.c, sp09/screen04.c, sp09/screen07.c, sp09/screen09.c,
--- a/testsuites/sptests/sp09/screen01.c
+++ b/testsuites/sptests/sp09/screen01.c
@@ -34,6 +34,16 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_delete - RTEMS_INVALID_ID" );

+  /* NULL return */
+  status = rtems_task_get_note( RTEMS_SELF, RTEMS_NOTEPAD_FIRST, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_get_note with NULL param"
+  );
+  puts( "TA1 - rtems_task_get_note - RTEMS_INVALID_ADDRESS" );
+
+  /* note too high */
  status = rtems_task_get_note( RTEMS_SELF, 100, &notepad_value );
  fatal_directive_status(
    status,
@@ -42,6 +52,7 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_get_note - RTEMS_INVALID_NUMBER" );

+  /* bad Id */
  status = rtems_task_get_note( 100, RTEMS_NOTEPAD_LAST, &notepad_value );
  fatal_directive_status(
    status,
@@ -50,6 +61,7 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_get_note - RTEMS_INVALID_ID" );

+  /* unused Id so invalid now */
  status = rtems_task_get_note(
    _RTEMS_tasks_Information.maximum_id,
    RTEMS_NOTEPAD_LAST,
@@ -90,7 +102,9 @@ void Screen1()
    RTEMS_NOTEPAD_LAST,
    &notepad_value
  );
-  fatal_directive_status( status, RTEMS_INVALID_ID,
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ID,
    "rtems_task_get_note with no tasks in API"
  );

@@ -103,6 +117,16 @@ void Screen1()
    "rtems_object_get_classic_name with no tasks in API"
  );

+  /* NULL param */
+  status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_ident NULL param"
+  );
+  puts( "TA1 - rtems_task_ident - RTEMS_INVALID_ADDRESS" );
+
+  /* OK */
  status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &self_id );
  directive_failed( status, "rtems_task_ident of self" );
  if ( self_id != Task_id[ 1 ] ) {
@@ -170,6 +194,16 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_resume - RTEMS_INCORRECT_STATE" );

+  /* NULL param */
+  status = rtems_task_set_priority( RTEMS_SELF, RTEMS_CURRENT_PRIORITY, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_set_priority with NULL param"
+  );
+  puts( "TA1 - rtems_task_set_priority - RTEMS_INVALID_ADDRESS" );
+
+  /* bad priority */
  status = rtems_task_set_priority( RTEMS_SELF, 512, &previous_priority );
  fatal_directive_status(
    status,
@@ -178,6 +212,7 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_set_priority - RTEMS_INVALID_PRIORITY" );

+  /* bad Id */
  status = rtems_task_set_priority( 100, 8, &previous_priority );
  fatal_directive_status(
    status,
@@ -214,7 +249,8 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_start - RTEMS_INVALID_ID" );

-  status = rtems_task_start( 0, Task_1, 0 );
+  /* already started */
+  status = rtems_task_start( RTEMS_SELF, Task_1, 0 );
  fatal_directive_status(
    status,
    RTEMS_INCORRECT_STATE,
@@ -222,6 +258,7 @@ void Screen1()
  );
  puts( "TA1 - rtems_task_start - RTEMS_INCORRECT_STATE" );

+  /* bad Id */
  status = rtems_task_suspend( 100 );
  fatal_directive_status(
    status,
@@ -229,4 +266,13 @@ void Screen1()
    "rtems_task_suspend with illegal id"
  );
  puts( "TA1 - rtems_task_suspend - RTEMS_INVALID_ID" );
+
+  /* NULL param */
+  status = rtems_task_mode( RTEMS_SELF, 0, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_mode with NULL param"
+  );
+  puts( "TA1 - rtems_task_mode - RTEMS_INVALID_ADDRESS" );
 }
--- a/testsuites/sptests/sp09/screen02.c
+++ b/testsuites/sptests/sp09/screen02.c
@@ -139,6 +139,16 @@ void Screen2()
  );
  puts( "TA1 - rtems_clock_get - RTEMS_INVALID_NUMBER" );

+  /* wake when NULL param */
+  status = rtems_task_wake_when( NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_NOT_DEFINED,
+    "rtems_task_wake_when NULL param"
+  );
+  puts( "TA1 - rtems_task_wake_when - RTEMS_INVALID_ADDRESS" );
+
+  /* wake when before set */
  status = rtems_task_wake_when( &time );
  if ( status == RTEMS_SUCCESSFUL ) {
    puts(
@@ -154,6 +164,7 @@ void Screen2()
    puts( "TA1 - rtems_task_wake_when - RTEMS_NOT_DEFINED" );
  }

+  /* before time set */
  status = rtems_timer_fire_when( 0, &time, Delayed_routine, NULL );
  if ( status == RTEMS_SUCCESSFUL ) {
    puts(
@@ -257,6 +268,16 @@ void Screen2()
  time.second += 3;
  puts( "TA1 - rtems_task_wake_when - TICKINVALID - sleep about 3 seconds" );

+  /* NULL time */
+  status = rtems_task_wake_when( NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_wake_when with NULL"
+  );
+  puts( "TA1 - rtems_task_wake_when - RTEMS_INVALID_ADDRESS" );
+
+  /* invalid ticks */
  status = rtems_task_wake_when( &time );
  directive_failed(
    status,
--- a/testsuites/sptests/sp09/screen03.c
+++ b/testsuites/sptests/sp09/screen03.c
@@ -24,6 +24,7 @@ void Screen3()
  rtems_status_code status;
  bool              skipUnsatisfied;

+  /* task create bad name */
  task_name = 1;
  status = rtems_task_create(
    0,
@@ -40,6 +41,22 @@ void Screen3()
  );
  puts( "TA1 - rtems_task_create - RTEMS_INVALID_NAME" );

+  /* null ID */
+  status = rtems_task_create(
+    Task_name[ 1 ],
+    4,
+    RTEMS_MINIMUM_STACK_SIZE,
+    RTEMS_DEFAULT_MODES,
+    RTEMS_DEFAULT_ATTRIBUTES,
+    NULL
+  );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_task_create with NULL ID param"
+  );
+  puts( "TA1 - rtems_task_create - RTEMS_INVALID_ADDRESS" );
+
  /*
   * If the bsp provides its own stack allocator, then
   * skip the test that tries to allocate a stack that is too big.
--- a/testsuites/sptests/sp09/screen05.c
+++ b/testsuites/sptests/sp09/screen05.c
@@ -22,6 +22,7 @@ void Screen5()
 {
  rtems_status_code status;

+  /* invalid name */
  status = rtems_semaphore_create(
    0,
    1,
@@ -36,6 +37,22 @@ void Screen5()
  );
  puts( "TA1 - rtems_semaphore_create - RTEMS_INVALID_NAME" );

+  /* NULL Id parameter */
+  status = rtems_semaphore_create(
+    Semaphore_name[ 1 ],
+    1,
+    RTEMS_DEFAULT_ATTRIBUTES,
+    RTEMS_NO_PRIORITY,
+    NULL
+  );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_semaphore_create with NULL param"
+  );
+  puts( "TA1 - rtems_semaphore_create - RTEMS_INVALID_ADDRESS" );
+
+  /* OK */
  status = rtems_semaphore_create(
    Semaphore_name[ 1 ],
    1,
--- a/testsuites/sptests/sp09/screen09.c
+++ b/testsuites/sptests/sp09/screen09.c
@@ -63,6 +63,7 @@ void Screen9()
    puts( "TA1 - rtems_interrupt_catch - old isr RTEMS_INVALID_ADDRESS" );
  #endif

+  /* send invalid id */
  status = rtems_signal_send( 100, RTEMS_SIGNAL_1 );
  fatal_directive_status(
    status,
@@ -71,6 +72,16 @@ void Screen9()
  );
  puts( "TA1 - rtems_signal_send - RTEMS_INVALID_ID" );

+  /* no signal in set */
+  status = rtems_signal_send( RTEMS_SELF, 0 );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_NUMBER,
+    "rtems_signal_send with no signals"
+  );
+  puts( "TA1 - rtems_signal_send - RTEMS_INVALID_NUMBER" );
+
+  /* no signal handler */
  status = rtems_signal_send( RTEMS_SELF, RTEMS_SIGNAL_16 );
  fatal_directive_status(
    status,
--- a/testsuites/sptests/sp09/screen14.c
+++ b/testsuites/sptests/sp09/screen14.c
@@ -25,6 +25,16 @@ void Screen14()
  rtems_timer_information timer_info;
  bool                    skipUnsatisfied;

+  /* NULL Id */
+  status = rtems_timer_create( Timer_name[ 1 ], NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_create NULL param"
+  );
+  puts( "TA1 - rtems_timer_create - RTEMS_INVALID_ADDRESS" );
+
+  /* bad name */
  status = rtems_timer_create( 0, &Junk_id );
  fatal_directive_status(
    status,
@@ -33,6 +43,7 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_create - RTEMS_INVALID_NAME" );

+  /* OK */
  status = rtems_timer_create( Timer_name[ 1 ], &Timer_id[ 1 ] );
  directive_failed( status, "rtems_timer_create" );
  puts( "TA1 - rtems_timer_create - 1 - RTEMS_SUCCESSFUL" );
@@ -93,6 +104,7 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_reset - RTEMS_NOT_DEFINED" );

+  /* bad id */
  status = rtems_timer_fire_after(
    rtems_build_id( 1, 1, 1, 256 ),
    5 * rtems_clock_get_ticks_per_second(),
@@ -106,6 +118,7 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_fire_after - RTEMS_INVALID_ID" );

+  /* bad id */
  build_time( &time, 12, 31, 1994, 9, 0, 0, 0 );
  status = rtems_timer_fire_when( 
    rtems_build_id( 1, 1, 1, 256 ),
@@ -120,6 +133,16 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_fire_when - RTEMS_INVALID_ID" );

+  /* NULL routine */
+  status = rtems_timer_fire_after( Timer_id[ 1 ], 1, NULL, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_fire_after with NULL handler"
+  );
+  puts( "TA1 - rtems_timer_fire_after - RTEMS_INVALID_ADDRESS" );
+
+  /* 0 ticks */
  status = rtems_timer_fire_after( Timer_id[ 1 ], 0, Delayed_routine, NULL );
  fatal_directive_status(
    status,
@@ -128,6 +151,16 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_fire_after - RTEMS_INVALID_NUMBER" );

+  /* NULL routine */
+  status = rtems_timer_fire_when( Timer_id[ 1 ], &time, NULL, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_fire_when with NULL handler"
+  );
+  puts( "TA1 - rtems_timer_fire_when - RTEMS_INVALID_ADDRESS" );
+  
+  /* invalid time -- before RTEMS epoch */ 
  build_time( &time, 2, 5, 1987, 8, 30, 45, 0 );
  status = rtems_timer_fire_when( Timer_id[ 1 ], &time, Delayed_routine, NULL );
  fatal_directive_status(
@@ -158,6 +191,16 @@ void Screen14()
    " - before RTEMS_INVALID_CLOCK\n"
  );

+  /* null param */
+  status = rtems_timer_get_information( Timer_id[ 1 ], NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_get_information with NULL param"
+  );
+  puts( "TA1 - rtems_timer_get_information - RTEMS_INVALID_ADDRESS" );
+
+  /* invalid id */
  status = rtems_timer_get_information( 100, &timer_info );
  fatal_directive_status(
    status,
@@ -168,6 +211,7 @@ void Screen14()

 /* timer server interface routines */

+  /* incorrect state */
  status = rtems_timer_server_fire_after( 0, 5, NULL, NULL );
  fatal_directive_status(
    status,
@@ -176,6 +220,7 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_server_fire_after - RTEMS_INCORRECT_STATE" );

+  /* incorrect state */
  status = rtems_timer_server_fire_when( 0, &time, NULL, NULL );
  fatal_directive_status(
    status,
@@ -184,8 +229,8 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_server_fire_when - RTEMS_INCORRECT_STATE" );

-  status =
-    rtems_timer_initiate_server( 0, 0, 0 );
+  /* invalid priority */
+  status = rtems_timer_initiate_server( 0, 0, 0 );
  fatal_directive_status(
    status,
    RTEMS_INVALID_PRIORITY,
@@ -218,6 +263,16 @@ void Screen14()
  directive_failed( status, "rtems_timer_initiate_server" );
  puts( "TA1 - rtems_timer_initiate_server - SUCCESSFUL" );

+  /* NULL routine */
+  status = rtems_timer_server_fire_after( Timer_id[ 1 ], 1, NULL, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_server_fire_after NULL routine"
+  );
+  puts( "TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ADDRESS" );
+
+  /* bad Id */
  status = rtems_timer_server_fire_after(
    rtems_build_id( 1, 1, 1, 256 ),
    5 * rtems_clock_get_ticks_per_second(),
@@ -231,6 +286,7 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ID" );

+  /* bad id */
  build_time( &time, 12, 31, 1994, 9, 0, 0, 0 );
  status = rtems_timer_server_fire_when(
    rtems_build_id( 1, 1, 1, 256 ),
@@ -245,7 +301,18 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_server_fire_when - RTEMS_INVALID_ID" );

-  status = rtems_timer_server_fire_after( Timer_id[ 1 ], 0, Delayed_routine, NULL );
+  /* NULL routine */
+  status = rtems_timer_server_fire_after( Timer_id[ 1 ], 1, NULL, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_server_fire_after NULL routine"
+  );
+  puts( "TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ADDRESS" );
+
+  /* 0 ticks */
+  status = rtems_timer_server_fire_after(
+    Timer_id[ 1 ], 0, Delayed_routine, NULL );
  fatal_directive_status(
    status,
    RTEMS_INVALID_NUMBER,
@@ -253,8 +320,10 @@ void Screen14()
  );
  puts( "TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_NUMBER" );

+  /* illegal time */
  build_time( &time, 2, 5, 1987, 8, 30, 45, 0 );
-  status = rtems_timer_server_fire_when( Timer_id[ 1 ], &time, Delayed_routine, NULL );
+  status = rtems_timer_server_fire_when(
+    Timer_id[ 1 ], &time, Delayed_routine, NULL );
  fatal_directive_status(
    status,
    RTEMS_INVALID_CLOCK,
@@ -270,6 +339,16 @@ void Screen14()
  directive_failed( status, "rtems_clock_get_tod" );
  print_time( "TA1 - rtems_clock_get_tod       - ", &time, "\n" );

+  /* when NULL routine */
+  status = rtems_timer_server_fire_when( Timer_id[ 1 ], &time, NULL, NULL );
+  fatal_directive_status(
+    status,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_timer_server_fire_when NULL routine"
+  );
+  puts( "TA1 - rtems_timer_server_fire_when - RTEMS_INVALID_ADDRESS" );
+
+  /* before current time */
  build_time( &time, 2, 5, 1990, 8, 30, 45, 0 );
  status = rtems_timer_server_fire_when(
    Timer_id[ 1 ], &time, Delayed_routine, NULL );
@@ -283,4 +362,5 @@ void Screen14()
    &time,
    " - before RTEMS_INVALID_CLOCK\n"
  );
+
 }
--- a/testsuites/sptests/sp09/sp09.scn
+++ b/testsuites/sptests/sp09/sp09.scn
@@ -2,11 +2,13 @@
 INIT - rtems_task_create - RTEMS_INVALID_PRIORITY
 INIT - rtems_task_restart - RTEMS_INCORRECT_STATE
 TA1 - rtems_task_delete - RTEMS_INVALID_ID
+TA1 - rtems_task_get_note - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_get_note - RTEMS_INVALID_NUMBER
 TA1 - rtems_task_get_note - RTEMS_INVALID_ID
 TA1 - rtems_task_get_note - RTEMS_INVALID_ID
 TA1 - rtems_task_get_note - RTEMS_INVALID_ID (no tasks in API)
 TA1 - rtems_object_get_classic_name - bad ID
+TA1 - rtems_task_ident - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_ident - current task RTEMS_SUCCESSFUL
 TA1 - rtems_task_ident - global RTEMS_INVALID_NAME
 TA1 - rtems_task_ident - local RTEMS_INVALID_NAME
@@ -14,6 +16,7 @@ TA1 - rtems_task_ident - RTEMS_INVALID_NODE
 TA1 - rtems_task_restart - RTEMS_INVALID_ID
 TA1 - rtems_task_resume - RTEMS_INVALID_ID
 TA1 - rtems_task_resume - RTEMS_INCORRECT_STATE
+TA1 - rtems_task_set_priority - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_set_priority - RTEMS_INVALID_PRIORITY
 TA1 - rtems_task_set_priority - RTEMS_INVALID_ID
 TA1 - rtems_task_set_note - RTEMS_INVALID_NUMBER
@@ -21,11 +24,12 @@ TA1 - rtems_task_set_note - RTEMS_INVALID_ID
 TA1 - rtems_task_start - RTEMS_INVALID_ID
 TA1 - rtems_task_start - RTEMS_INCORRECT_STATE
 TA1 - rtems_task_suspend - RTEMS_INVALID_ID
+TA1 - rtems_task_mode - RTEMS_INVALID_ADDRESS
 <pause - screen 2>
 TA1 - rtems_clock_get_tod - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_get_tod - RTEMS_NOT_DEFINED
 TA1 - rtems_clock_get_seconds_since_epoch - RTEMS_INVALID_ADDRESS
-TA1 - rtems_clock_get_second_since_epoch - RTEMS_NOT_DEFINED
+TA1 - rtems_clock_get_seconds_since_epoch - RTEMS_NOT_DEFINED
 TA1 - rtems_clock_get_uptime - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_get_tod_timeval - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_get_tod_timeval - RTEMS_NOT_DEFINED
@@ -33,6 +37,7 @@ TA1 - rtems_clock_set_nanoseconds_extension - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_set - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_get - RTEMS_INVALID_ADDRESS
 TA1 - rtems_clock_get - RTEMS_INVALID_NUMBER
+TA1 - rtems_task_wake_when - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_wake_when - RTEMS_NOT_DEFINED
 TA1 - rtems_timer_fire_when - RTEMS_NOT_DEFINED
 TA1 - rtems_clock_set - 08:30:45   02/05/1987 - RTEMS_INVALID_CLOCK
@@ -44,6 +49,7 @@ TA1 - rtems_clock_set - 08:30:61   02/05/1988 - RTEMS_INVALID_CLOCK
 TA1 - rtems_clock_set - 08:30:45   02/05/1988 - RTEMS_INVALID_CLOCK
 TA1 - rtems_clock_set - 08:30:45   02/05/1988 - RTEMS_SUCCESSFUL
 TA1 - rtems_task_wake_when - TICKINVALID - sleep about 3 seconds
+TA1 - rtems_task_wake_when - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_wake_when - TICKINVALID - woke up RTEMS_SUCCESSFUL
 TA1 - rtems_task_wake_when - 08:30:48   02/05/1961 - RTEMS_INVALID_CLOCK
 TA1 - rtems_task_wake_when - 25:30:48   02/05/1988 - RTEMS_INVALID_CLOCK
@@ -51,6 +57,7 @@ TA1 - current time - 08:30:51   02/05/1988
 TA1 - rtems_task_wake_when - 08:30:51   01/05/1988 - RTEMS_INVALID_CLOCK
 <pause - screen 3>
 TA1 - rtems_task_create - RTEMS_INVALID_NAME
+TA1 - rtems_task_create - RTEMS_INVALID_ADDRESS
 TA1 - rtems_task_create - stack size - RTEMS_UNSATISFIED
 TA1 - rtems_task_create - TA2 created - RTEMS_SUCCESSFUL
 TA1 - rtems_task_suspend - suspend TA2 - RTEMS_SUCCESSFUL
@@ -76,8 +83,10 @@ TA1 - rtems_event_send - RTEMS_INVALID_ID
 TA1 - rtems_task_wake_after - sleep 1 second - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_set - 08:30:45   02/05/1988 - RTEMS_SUCCESSFUL
 TA1 - current time - Fri Feb  5 08:30:45 1988
+
 <pause - screen 5>
 TA1 - rtems_semaphore_create - RTEMS_INVALID_NAME
+TA1 - rtems_semaphore_create - RTEMS_INVALID_ADDRESS
 TA1 - rtems_semaphore_create - 1 - RTEMS_SUCCESSFUL
 TA1 - rtems_semaphore_create - 2 - RTEMS_SUCCESSFUL
 TA1 - rtems_semaphore_create - 3 - RTEMS_TOO_MANY
@@ -126,6 +135,7 @@ TA1 - rtems_message_queue_delete - local RTEMS_INVALID_ID
 TA1 - rtems_message_queue_ident - RTEMS_INVALID_NAME
 TA1 - rtems_message_queue_get_number_pending - RTEMS_INVALID_ADDRESS
 TA1 - rtems_message_queue_get_number_pending - RTEMS_INVALID_ID
+TA1 - rtems_message_queue_flush - RTEMS_INVALID_ADDRESS
 TA1 - rtems_message_queue_flush - RTEMS_INVALID_ID
 TA1 - rtems_message_queue_receive - RTEMS_INVALID_ID
 TA1 - rtems_message_queue_receive - Q 1 - RTEMS_INVALID_ADDRESS NULL buffer
@@ -133,12 +143,12 @@ TA1 - rtems_message_queue_receive - Q 1 - RTEMS_INVALID_ADDRESS NULL size
 TA1 - rtems_message_queue_receive - Q 1 - RTEMS_UNSATISFIED
 TA1 - rtems_message_queue_receive - Q 1 - timeout in 3 seconds
 TA1 - rtems_message_queue_receive - Q 1 - woke up with RTEMS_TIMEOUT
-TA1 - rtems_message_queue_send - RTEMS_INVALID_ADDRESS
+TA1 - rtems_message_queue_send - NULL buffer - RTEMS_INVALID_ADDRESS
 TA1 - rtems_message_queue_send - RTEMS_INVALID_ID
 TA1 - rtems_message_queue_send - BUFFER 1 TO Q 1 - RTEMS_SUCCESSFUL
 TA1 - rtems_message_queue_send - BUFFER 2 TO Q 1 - RTEMS_SUCCESSFUL
 TA1 - rtems_message_queue_send - BUFFER 3 TO Q 1 - RTEMS_TOO_MANY
-TA1 - rtems_message_queue_urgent - RTEMS_INVALID_ADDRESS
+TA1 - rtems_message_queue_urgent - NULL buffer - RTEMS_INVALID_ADDRESS
 TA1 - rtems_message_queue_urgent - RTEMS_INVALID_ID
 TA1 - rtems_message_queue_broadcast - NULL buffer - RTEMS_INVALID_ADDRESS
 TA1 - rtems_message_queue_broadcast - NULL count - RTEMS_INVALID_ADDRESS
@@ -168,6 +178,7 @@ TA1 - rtems_interrupt_catch - RTEMS_INVALID_NUMBER
 TA1 - rtems_interrupt_catch - bad handler RTEMS_INVALID_ADDRESS
 TA1 - rtems_interrupt_catch - old isr RTEMS_INVALID_ADDRESS
 TA1 - rtems_signal_send - RTEMS_INVALID_ID
+TA1 - rtems_signal_send - RTEMS_INVALID_NUMBER
 TA1 - rtems_signal_send - RTEMS_NOT_DEFINED
 TA1 - rtems_port_create - RTEMS_INVALID_NAME
 TA1 - rtems_port_create - bad range - RTEMS_INVALID_ADDRESS
@@ -175,10 +186,9 @@ TA1 - rtems_port_create - null id - RTEMS_INVALID_ADDRESS
 TA1 - rtems_port_create - RTEMS_TOO_MANY
 TA1 - rtems_port_delete - RTEMS_INVALID_ID
 TA1 - rtems_port_ident - RTEMS_INVALID_NAME
-TA1 - rtems_port_external_to_internal - RTEMS_INVALID_ID
 TA1 - rtems_port_external_to_internal - RTEMS_INVALID_ADDRESS
 TA1 - rtems_port_internal_to_external - RTEMS_INVALID_ID
-TA1 - rtems_port_internal_to_external - RTEMS_INVALID_ADDRESS
+TA1 - rtems_port_external_to_internal - RTEMS_INVALID_ADDRESS
 <pause - screen 10>
 TA1 - rtems_rate_monotonic_create - RTEMS_INVALID_ADDRESS
 TA1 - rtems_rate_monotonic_create - RTEMS_INVALID_NAME
@@ -250,7 +260,6 @@ TA1 - rtems_region_get_segment - RTEMS_INVALID_ID
 TA1 - rtems_region_get_segment - RTEMS_INVALID_ADDRESS
 TA1 - rtems_region_get_segment - 0 size - RTEMS_INVALID_SIZE
 TA1 - rtems_region_get_segment - too big - RTEMS_INVALID_SIZE
-TA1 - rtems_region_get_segment - RTEMS_INVALID_SIZE
 TA1 - rtems_region_get_segment - RTEMS_SUCCESSFUL
 TA1 - rtems_region_get_segment - RTEMS_UNSATISFIED
 TA1 - rtems_region_get_segment - timeout in 3 seconds
@@ -283,7 +292,7 @@ TA1 - rtems_io_write - RTEMS_INVALID_NUMBER
 TA1 - rtems_clock_set - 23:59:59   12/31/2000 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_get_tod - 00:00:00   01/01/2001 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_set - 23:59:59   12/31/1999 - RTEMS_SUCCESSFUL
-TA1 - rtems_clock_get_tod - 00:00:00   01/01/2000 - RTEMS_SUCCESSFUL
+TA1 - rtems_clock_get - 00:00:00   01/01/2000 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_set - 23:59:59   12/31/2100 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_get_tod - 17:31:44   11/25/1964 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_set - 23:59:59   12/31/2099 - RTEMS_SUCCESSFUL
@@ -291,6 +300,7 @@ TA1 - rtems_clock_get_tod - 17:31:44   11/25/1963 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_set - 23:59:59   12/31/1991 - RTEMS_SUCCESSFUL
 TA1 - rtems_clock_get_tod - 00:00:00   01/01/1992 - RTEMS_SUCCESSFUL
 <pause - screen 14>
+TA1 - rtems_timer_create - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_create - RTEMS_INVALID_NAME
 TA1 - rtems_timer_create - 1 - RTEMS_SUCCESSFUL
 TA1 - rtems_timer_create - 2 - RTEMS_TOO_MANY
@@ -302,20 +312,26 @@ TA1 - rtems_timer_reset - RTEMS_INVALID_ID
 TA1 - rtems_timer_reset - RTEMS_NOT_DEFINED
 TA1 - rtems_timer_fire_after - RTEMS_INVALID_ID
 TA1 - rtems_timer_fire_when - RTEMS_INVALID_ID
+TA1 - rtems_timer_fire_after - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_fire_after - RTEMS_INVALID_NUMBER
+TA1 - rtems_timer_fire_when - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_fire_when - 08:30:45   02/05/1987 - RTEMS_INVALID_CLOCK
 TA1 - rtems_clock_get_tod       - 00:00:00   01/01/1992
 TA1 - rtems_timer_fire_when - 08:30:45   02/05/1990 - before RTEMS_INVALID_CLOCK
+TA1 - rtems_timer_get_information - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_get_information - RTEMS_INVALID_ID
 TA1 - rtems_timer_server_fire_after - RTEMS_INCORRECT_STATE
 TA1 - rtems_timer_server_fire_when - RTEMS_INCORRECT_STATE
 TA1 - rtems_timer_initiate_server - RTEMS_INVALID_PRIORITY
 TA1 - rtems_timer_initiate_server - RTEMS_UNSATISFIED
 TA1 - rtems_timer_initiate_server - SUCCESSFUL
+TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ID
 TA1 - rtems_timer_server_fire_when - RTEMS_INVALID_ID
+TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_server_fire_after - RTEMS_INVALID_NUMBER
 TA1 - rtems_timer_server_fire_when - 08:30:45   02/05/1987 - RTEMS_INVALID_CLOCK
 TA1 - rtems_clock_get_tod       - 00:00:00   01/01/1992
+TA1 - rtems_timer_server_fire_when - RTEMS_INVALID_ADDRESS
 TA1 - rtems_timer_server_fire_when - 08:30:45   02/05/1990 - before RTEMS_INVALID_CLOCK
 *** END OF TEST 9 ***
--- a/testsuites/sptests/sp28/init.c
+++ b/testsuites/sptests/sp28/init.c
@@ -115,6 +115,13 @@ void test_errors(void)
  rtems_status_code  sc;
  void              *value;

+  /*
+   *  task variable add error status codes
+   */
+  puts( "task variable add - NULL pointer - RTEMS_INVALID_ADDRESS" );
+  sc = rtems_task_variable_get(RTEMS_SELF, NULL, NULL );
+  fatal_directive_status( sc, RTEMS_INVALID_ADDRESS, "add NULL pointer" );
+
  /*
   *  task variable get error status codes
   */
--- a/testsuites/sptests/sp28/sp28.scn
+++ b/testsuites/sptests/sp28/sp28.scn
@@ -1,4 +1,5 @@
 *** START OF TEST 28 ***
+task variable add - NULL pointer - RTEMS_INVALID_ADDRESS
 task variable get - bad Id - RTEMS_INVALID_ID
 task variable get - NULL pointer - RTEMS_INVALID_ADDRESS
 task variable get - bad result - RTEMS_INVALID_ADDRESS
--- a/testsuites/sptests/sp43/init.c
+++ b/testsuites/sptests/sp43/init.c
@@ -201,6 +201,14 @@ rtems_task Init(
   * rtems_object_set_name - errors
   */

+  puts( "rtems_object_set_name - INVALID_ADDRESS" );
+  sc = rtems_object_set_name( tmpId, NULL );
+  fatal_directive_status(
+    sc,
+    RTEMS_INVALID_ADDRESS,
+    "rtems_object_set_name INVALID_ADDRESS"
+  );
+
  puts( "rtems_object_set_name - INVALID_ID (bad API)" );
  tmpId = rtems_build_id( 0xff, OBJECTS_RTEMS_TASKS, 1, 1 ),
  sc = rtems_object_set_name( tmpId, newName );