解放されたインスタンスに送信されたメッセージ... @合成中に送信されましたか？

Question

私はRaphael CruzeiroのPDF Annotatorのコードを使用していて、数多くのメモリリークを発見しました（ARCはオフになっており、古いデバイスをサポートするためには使用できません）。それらのほとんどをパッチした後、私は最後のカップルに行って、彼らは私を困惑させた。したがって、PDFDocumentというクラスでは、彼はCGPDFPageRef,CGPDFDocument、カスタムアノテーションクラス@synthesize 'dのプロパティを持っています。私は、リリースでdeallocメソッドをペーストしなければならなかったし、小さな問題を除いてうまくいくいくつかのぶら下がりポインタを排除しなければならなかった：約3つの完全な解放サイクルの後、@ annhesationオブジェクトの@synthesize行でクラッシュする... @synthesize中に送信された割り当て解除されたオブジェクトのためにSIGABRTを見たことはないので、自然にそれを修正する方法は知られていません。 deallocでリリースコードを削除した場合はリークしますが、残しておくとクラッシュします。ここでPDFDocumentクラスのコードだ：

//.h 

#import <Foundation/Foundation.h> 

@class Annotation; 

@interface PDFDocument : NSObject { 
    Annotation *_annotation; 
} 

- (id)initWithDocument:(NSString *)documentPath; 

- (NSInteger) pageCount; 
- (void) loadPage:(NSInteger)number; 
- (BOOL)save; 

@property (nonatomic, retain) NSString *name; 
@property (nonatomic, retain) NSString *hash; 
@property (readwrite, nonatomic, assign) CGPDFDocumentRef document; 
@property (readwrite, nonatomic, assign) CGPDFPageRef page; 

@property (nonatomic, retain) NSString *version; 

@property (nonatomic, assign) BOOL dirty; 

@property (nonatomic, retain) Annotation *annotation; 

@end 

//.m 
#import "PDFDocument.h" 
#import "Annotation.h" 
#import "HashExtensions.h" 
#import "DocumentDeserializer.h" 
#import "DocumentSerializer.h" 


@implementation PDFDocument 

@synthesize document; 
@synthesize page; 
@synthesize annotation = _annotation; //after 3rd cycle, it crashes here. 
@synthesize name; 
@synthesize hash; 
@synthesize dirty; 
@synthesize version; 

- (id)initWithDocument:(NSString *)documentPath 
{ 
    if((self = [super init]) != NULL) { 

     self.name = [documentPath lastPathComponent]; 
     if ([self.name isEqualToString:@"Musette.pdf"] || [self.name isEqualToString:@"Minore.pdf"] || [self.name isEqualToString:@"Cantata.pdf"] || [self.name isEqualToString:@"Finalé.pdf"]) 
     { 
     CFURLRef ref = CFBundleCopyResourceURL(CFBundleGetMainBundle(), (CFStringRef)self.name, NULL, NULL); 
     self.document = CGPDFDocumentCreateWithURL(ref); 
     self.page = CGPDFDocumentGetPage(document, 1); 
     self.version = @"1.0"; 
     DocumentDeserializer *deserializer = [[[DocumentDeserializer alloc] init] autorelease]; 
     self.annotation = [deserializer readAnnotation:[[(NSURL*)ref absoluteString] stringByDeletingPathExtension]]; 

     CFRelease(ref); 
     } 

     else { 

      CFURLRef pdfURL = (CFURLRef)[[NSURL alloc] initFileURLWithPath:documentPath]; 
      self.document = CGPDFDocumentCreateWithURL(pdfURL); 
      self.page = CGPDFDocumentGetPage(document, 1); 
      self.version = @"1.0"; 
      DocumentDeserializer *deserializer = [[[DocumentDeserializer alloc] init] autorelease]; 
      self.annotation = [deserializer readAnnotation:[[(NSURL*)pdfURL absoluteString] stringByDeletingPathExtension]]; 

      CFRelease(pdfURL); 
      CGPDFPageRelease(self.page); 

     } 
    } 

    return self; 
} 

- (NSInteger)pageCount 
{ 
    return CGPDFDocumentGetNumberOfPages(self.document); 
} 

- (void)loadPage:(NSInteger)number 
{ 
    self.page = CGPDFDocumentGetPage(document, number); 
} 

- (BOOL)save 
{ 
    DocumentSerializer *serializer = [[[DocumentSerializer alloc] init] autorelease]; 
    [serializer serialize:self]; 

    self.dirty = NO; 
    return !self.dirty; 
} 

- (void)dealloc 
{ 
    CGPDFDocumentRelease(self.document); 
    if (self.annotation != nil && _annotation != nil) { 
     [_annotation release]; 
     self.annotation = nil; 
    } //my attempt to prevent the object from being over-released 
    self.document = nil; 
    self.name = nil; 
    [super dealloc]; 
} 

@end 

それから私はゾンビのオブジェクトを見つけるために、楽器を通してそれを走り、案の定、楽器はまったく同じ@synthesizeラインでメッセージを送信される割り当てが解除されたオブジェクトを発見しました！

誰でも何が起こっているのか、それを修正する方法はありますか？

Answer 1

このビットは、非常に間違っになります

if (self.annotation != nil && _annotation != nil) { 
    [_annotation release]; 
    self.annotation = nil; 
} 

を第一に、なぜあなたがnilネスのためのself.annotationと_annotationをチェックしています。これは効果的に同じチェックを2回実行しています。

第二に、あなたは_annotationを解放するために直接IVARのアクセスを使用していて、その後、annotationのためのセッターは再び_annotationを解放し、_annotation = nilを設定します。効果的には、このやっている：あなたが見ることができるよう

if (self.annotation != nil && _annotation != nil) { 
    [_annotation release]; 
    [_annotation release]; 
    _annotation = [nil retain]; 
} 

を、過リリース_annotationに起こっています。

また、真剣に、ARCを使用してください。 ARCは（主に）コンパイル時に実行されているデバイスやOSのバージョンとは関係ありません。 pre iOS 5でサポートされていない唯一のビットは、自動ニルエイド弱ポインタ​​です。とにかくそれはLion/iOS 5でまったく新しくなっているので、本当に問題ではありません。