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.

11 files changed, 244 insertions, 13 deletions

diff --git a/testsuites/sptests/ChangeLog b/testsuites/sptests/ChangeLog
index 7951063dc7..b8e615f068 100644
--- a/testsuites/sptests/ChangeLog
+++ b/testsuites/sptests/ChangeLog
@@ -1,5 +1,12 @@
 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,
 	sp09/screen10.c, sp09/screen11.c, sp09/screen12.c, sp09/sp09.scn: Add
 	some error cases uncovered by uC5282 coverage runs.
diff --git a/testsuites/sptests/sp09/screen01.c b/testsuites/sptests/sp09/screen01.c
index 7edd55ef2d..25b41eb5eb 100644
--- 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" );
 }
diff --git a/testsuites/sptests/sp09/screen02.c b/testsuites/sptests/sp09/screen02.c
index 1fa33c03a0..4f14c632be 100644
--- 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,
diff --git a/testsuites/sptests/sp09/screen03.c b/testsuites/sptests/sp09/screen03.c
index 7381bd6cb6..993bc8b479 100644
--- 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.
diff --git a/testsuites/sptests/sp09/screen05.c b/testsuites/sptests/sp09/screen05.c
index 3747de44af..08f92781aa 100644
--- 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,
diff --git a/testsuites/sptests/sp09/screen09.c b/testsuites/sptests/sp09/screen09.c
index dfd725d2d7..5d50ec9667 100644
--- 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,
diff --git a/testsuites/sptests/sp09/screen14.c b/testsuites/sptests/sp09/screen14.c
index acdf5da074..32d0379168 100644
--- 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"
   );
+
 }
diff --git a/testsuites/sptests/sp09/sp09.scn b/testsuites/sptests/sp09/sp09.scn
index c93135f866..b976bcdbec 100644
--- 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 ***
diff --git a/testsuites/sptests/sp28/init.c b/testsuites/sptests/sp28/init.c
index 4abbada522..77009b14b1 100644
--- a/testsuites/sptests/sp28/init.c
+++ b/testsuites/sptests/sp28/init.c
@@ -116,6 +116,13 @@ void test_errors(void)
   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
    */
   puts( "task variable get - bad Id - RTEMS_INVALID_ID" );
diff --git a/testsuites/sptests/sp28/sp28.scn b/testsuites/sptests/sp28/sp28.scn
index ae90fbb161..654ae97d1e 100644
--- 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
diff --git a/testsuites/sptests/sp43/init.c b/testsuites/sptests/sp43/init.c
index 19c6c896b4..e7f4f5d9f3 100644
--- 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 );