gnustep/libs-base
0
0
Fork 0
mirror of https://github.com/gnustep/libs-base.git synced 2025-04-22 16:33:29 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix the recursive lock bug.

Browse source 
git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@17171 72102866-910b-0410-8b05-ffd578937521
This commit is contained in:
Alexander Malmberg 2003-07-08 01:10:48 +00:00
parent 76568f1f5c
commit 568aca9d71
2 changed files with 24 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
ChangeLog
View file

@ -1,3 +1,12 @@
2003-07-08 03:03 Alexander Malmberg <alexander@malmberg.org>
* Source/NSConnection.m (-locateLocalTarget:): The targetToCached
map contains CachedLocalObject:s, not GSLocalCounter:s. Extract the
GSLocalCounter from the CachedLocalObject.
Unlock the global_proxies_gate lock before calling
-proxyWithLocal:connection:.
2003-07-07 David Ayers <d.ayers@inode.at>
* Headers/gnustep/base/GSObjCRuntime.h:

20
Source/NSConnection.m
View file

@ -3209,6 +3209,7 @@ static void callEncoder (DOContext *ctxt)
if (node != 0)
{
proxy = node->value.obj;
M_UNLOCK(global_proxies_gate);
}
else
{
@ -3217,17 +3218,24 @@ static void callEncoder (DOContext *ctxt)
counter = NSMapGet (targetToCounter, (void*)target);
if (counter == nil)
{
CachedLocalObject *cached;
/*
* If the target doesn't exist for any connection, but still
* persists in the cache (ie it was recently released) then
* we move it back from the cache to the main maps so we can
* retain it on this connection.
*/
counter = NSMapGet (targetToCached, (void*)target);
if (counter != nil)
cached = NSMapGet (targetToCached, (void*)target);
if (cached != nil)
{
unsigned t = counter->target;
id o = counter->object;
unsigned t;
id o;
counter = [cached obj];
t = counter->target;
o = counter->object;
NSMapInsert(objectToCounter, (void*)o, counter);
NSMapInsert(targetToCounter, (void*)t, counter);
@ -3237,6 +3245,7 @@ static void callEncoder (DOContext *ctxt)
}
}
RETAIN(counter);
M_UNLOCK(global_proxies_gate);
if (counter == nil)
{
if(debug_connection > 3)
@ -3244,6 +3253,8 @@ static void callEncoder (DOContext *ctxt)
}
else
{
NSAssert([counter isKindOfClass: [GSLocalCounter class]],
@"Local counter is wrong kind of class.");
proxy = [distantObjectClass proxyWithLocal: counter->object
connection: self];
if (debug_connection > 3)
@ -3252,7 +3263,6 @@ static void callEncoder (DOContext *ctxt)
RELEASE(counter);
}
}
M_UNLOCK(global_proxies_gate);
return proxy;
}