gnustep/libs-base
0
0
Fork 0
mirror of https://github.com/gnustep/libs-base.git synced 2025-04-23 00:41:02 +00:00
Code Issues Projects Releases Packages Wiki Activity

fix for bug # 44240

Browse source 
git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@38340 72102866-910b-0410-8b05-ffd578937521
This commit is contained in:
Richard Frith-MacDonald 2015-02-14 05:50:48 +00:00
parent 1fa5505332
commit 9fcfab1a93
2 changed files with 30 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
ChangeLog
View file

@ -1,3 +1,10 @@
2015-02-14 Richard Frith-Macdonald <rfm@gnu.org>
* Source/NSThread.m: Be more agressive about releasing thread
resources when a thread exists (in particular release the
thread's runloop repeatedly for bug #44240) in case of the
deallocation of objects causing re-population of thread data.
2015-02-07 Richard Frith-Macdonald <rfm@gnu.org>
* Source/NSData.m: Simplify mingw/unixy ifdefs for writing a file.

41
Source/NSThread.m
View file

@ -710,18 +710,14 @@ unregisterActiveThread(NSThread *thread)
- (void) dealloc
{
int retries = 0;
if (_active == YES)
{
[NSException raise: NSInternalInconsistencyException
format: @"Deallocating an active thread without [+exit]!"];
}
if (_runLoopInfo != 0)
{
GSRunLoopThreadInfo *info = (GSRunLoopThreadInfo*)_runLoopInfo;
_runLoopInfo = 0;
[info release];
}
DESTROY(_runLoopInfo);
DESTROY(_thread_dictionary);
DESTROY(_target);
DESTROY(_arg);
@ -731,24 +727,33 @@ unregisterActiveThread(NSThread *thread)
[NSAutoreleasePool _endThread: self];
}
if (_thread_dictionary != nil)
while ((_thread_dictionary != nil || _runLoopInfo != nil) && retries++ < 10)
{
/*
* Try again to get rid of thread dictionary.
/* Try again.
*/
DESTROY(_runLoopInfo);
DESTROY(_thread_dictionary);
if (_autorelease_vars.pool_cache != 0)
{
[NSAutoreleasePool _endThread: self];
}
if (_thread_dictionary != nil)
{
NSLog(@"Oops - leak - thread dictionary is %@", _thread_dictionary);
if (_autorelease_vars.pool_cache != 0)
{
[NSAutoreleasePool _endThread: self];
}
}
}
if (_runLoopInfo != nil)
{
NSLog(@"Oops - leak - run loop is %@", _runLoopInfo);
if (_autorelease_vars.pool_cache != 0)
{
[NSAutoreleasePool _endThread: self];
}
}
if (_thread_dictionary != nil)
{
NSLog(@"Oops - leak - thread dictionary is %@", _thread_dictionary);
if (_autorelease_vars.pool_cache != 0)
{
[NSAutoreleasePool _endThread: self];
}
}
DESTROY(_gcontext);
[super dealloc];