diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2020-09-11 15:43:23 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2020-09-11 15:47:02 +0200 |
commit | dd734d467d6e0961a88bdabbafccede6cccb2156 (patch) | |
tree | a47e332db77339a932abe10bd34a71343737d652 | |
parent | e563e61c7aa22e87d5d0631011b80622be5b4557 (diff) |
score: Fix _Thread_Initialize()
Fix an error cleanup path in SMP configurations to avoid a NULL pointer access.
Update #3959.
-rw-r--r-- | cpukit/score/src/threadinitialize.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/cpukit/score/src/threadinitialize.c b/cpukit/score/src/threadinitialize.c index 4a74426498..4c6d849111 100644 --- a/cpukit/score/src/threadinitialize.c +++ b/cpukit/score/src/threadinitialize.c @@ -60,7 +60,6 @@ bool _Thread_Initialize( } /* Set everything to perform the error case clean up */ - scheduler_index = 0; the_thread->Start.stack_free = config->stack_free; #if defined(RTEMS_SMP) @@ -153,6 +152,7 @@ bool _Thread_Initialize( scheduler_node = NULL; scheduler_node_for_index = the_thread->Scheduler.nodes; scheduler_for_index = &_Scheduler_Table[ 0 ]; + scheduler_index = 0; while ( scheduler_index < _Scheduler_Count ) { Priority_Control priority_for_index; @@ -255,10 +255,6 @@ bool _Thread_Initialize( if ( extension_status ) return true; -#if defined(RTEMS_SMP) || CPU_ENABLE_ROBUST_THREAD_DISPATCH == TRUE -failed: -#endif - #if defined(RTEMS_SMP) while ( scheduler_index > 0 ) { scheduler_node_for_index = (Scheduler_Node *) @@ -277,6 +273,11 @@ failed: &information->Thread_queue_heads.Free, the_thread->Wait.spare_heads ); + +#if defined(RTEMS_SMP) || CPU_ENABLE_ROBUST_THREAD_DISPATCH == TRUE +failed: +#endif + ( *the_thread->Start.stack_free )( the_thread->Start.Initial_stack.area ); return false; } |