[Author: dmaclach]

Subtle issue in GTMLightweightProxy where it would leak (and return nil) if 
you initialized it using a standard alloc] init] pattern instead if
initWithRepresentedObject:]. The init call would go to the proxy which
being nil would return nil, and you'd end up with both a leak and a nil pointer
nasty.

R=thomasvl
DELTA=13  (13 added, 0 deleted, 0 changed)

1 files changed, 9 insertions, 0 deletions

diff --git a/Foundation/GTMLightweightProxyTest.m b/Foundation/GTMLightweightProxyTest.m
index e50a9dc..01a1bb2 100644
--- a/Foundation/GTMLightweightProxyTest.m
+++ b/Foundation/GTMLightweightProxyTest.m
@@ -30,6 +30,15 @@
 
 @implementation GTMLightweightProxyTest
 
+- (void)testInit {
+  id proxy = [[[GTMLightweightProxy alloc] 
+               initWithRepresentedObject:self] autorelease];
+  STAssertNotNil(proxy, nil);
+  
+  proxy = [[[GTMLightweightProxy alloc] init] autorelease];
+  STAssertNotNil(proxy, nil);
+}
+
 - (void)testProxy {
   id proxy
     = [[[GTMLightweightProxy alloc] initWithRepresentedObject:self] autorelease];