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/wkspace.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/wkspace.c')
-rw-r--r-- | cpukit/score/src/wkspace.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/cpukit/score/src/wkspace.c b/cpukit/score/src/wkspace.c index 887fa77b45..f8994c2e8d 100644 --- a/cpukit/score/src/wkspace.c +++ b/cpukit/score/src/wkspace.c @@ -79,7 +79,7 @@ void *_Workspace_Allocate( /* * _Workspace_Free */ -bool _Workspace_Free( +void _Workspace_Free( void *block ) { @@ -91,7 +91,8 @@ bool _Workspace_Free( __builtin_return_address( 1 ) ); #endif - return _Heap_Free( &_Workspace_Area, block ); + if (block) + _Heap_Free( &_Workspace_Area, block ); } /* |