diff options
| author | Joel Sherrill <joel.sherrill@OARcorp.com> | 2011-02-16 00:24:49 +0000 |
|---|---|---|
| committer | Joel Sherrill <joel.sherrill@OARcorp.com> | 2011-02-16 00:24:49 +0000 |
| commit | a0323a9f8f673f2b4d7a70e2019fa47d3df76d96 (patch) | |
| tree | 86f72a2427700fa35af175738c9efb404fae2e96 /cpukit/score/src/threadclose.c | |
| parent | 2011-02-15 Jennifer Averett (diff) | |
| download | rtems-a0323a9f8f673f2b4d7a70e2019fa47d3df76d96.tar.bz2 | |
2011-02-15 Joel Sherrill <joel.sherrilL@OARcorp.com>
* libmisc/capture/capture.c, posix/src/keyfreememory.c,
posix/src/pthread.c, score/include/rtems/score/wkspace.h,
score/src/objectextendinformation.c,
score/src/objectnamespaceremove.c, score/src/objectsetname.c,
score/src/threadclose.c, score/src/threadinitialize.c,
score/src/wkspace.c: Many places were checking for a NULL pointer
before calling _Workspace_Free. By moving the check into
_Workspace_Free, we eliminate a number of conditional paths and make
it harder to return a NULL pointer.
Diffstat (limited to 'cpukit/score/src/threadclose.c')
| -rw-r--r-- | cpukit/score/src/threadclose.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/cpukit/score/src/threadclose.c b/cpukit/score/src/threadclose.c index aa29ae6a21..2ba29519d9 100644 --- a/cpukit/score/src/threadclose.c +++ b/cpukit/score/src/threadclose.c @@ -101,8 +101,7 @@ void _Thread_Close( #endif the_thread->fp_context = NULL; - if ( the_thread->Start.fp_context ) - (void) _Workspace_Free( the_thread->Start.fp_context ); + _Workspace_Free( the_thread->Start.fp_context ); #endif /* @@ -112,7 +111,6 @@ void _Thread_Close( _Thread_Stack_Free( the_thread ); the_thread->Start.stack = NULL; - if ( the_thread->extensions ) - (void) _Workspace_Free( the_thread->extensions ); + _Workspace_Free( the_thread->extensions ); the_thread->extensions = NULL; } |