NSMutableStringデータ追加エラー

Question

私はこのようなAsyncSocketを持っています。

- (void)onSocket:(AsyncSocket *)sock didReadData:(NSData *)data withTag:(long)tag { 
NSString *message = [[[NSString alloc] initWithData:data encoding:NSUTF8StringEncoding] autorelease]; 
[data getBytes:&tdata]; 
    if (tdata > 5) { 

    if(bheader){ 
     if(!charS){ 
      if([message isEqualToString:@"S"]){ 
       CMSG = message; 
       charS=YES; 
      } 
     } 
     else{ 
      NSMutableString *tmp = [[NSMutableString alloc] initWithString:@""]; 
      [tmp appendString:CMSG];  <<<<< This is code error at loop 2, 
      [tmp appendString:message];  the first loop success but second is fail 
      CMSG = tmp; 
      [tmp release]; 
     } 
    } 
    else{ 
     if (message){    
      cmessage = [[NSString alloc]initWithFormat:@"%@%@",cmessage,message] ; 
     } 
     else 
      NSLog(@"Error converting received data into UTF-8 String"); 

     cdata++; 
     if(cdata==idata) { 
      msgComplete=YES; 
     } 

    } 

    if (msgComplete) { 
     NSDictionary *userInfo = [NSDictionary dictionaryWithObject:cmessage forKey:kNotificationMessage]; 
     [notificationCenter postNotificationName:kNotification object:self userInfo:userInfo]; 
     cmessage=@""; 
     CMSG=@""; 
     msgComplete=NO; 
     bheader=YES; 
     cdata=0; 
     charS=NO; 
     [cmessage release]; 

    } 
} 

[sock readDataToLength:1 withTimeout:-1 tag:0]; 

}

このコードは[tmp appendString:CMSG];スレッド1の第2ループで失敗：プログラムは、信号 "SIGABRT" を受信
このエラーを修正する方法は？
ありがとうございます。

Answer 1

CMSGはtmpに割り当てられましたが、直後にtmpが解放されます。 CMSGをループ全体に保持する必要があります。

CMSG = tmp; //<< should be CMSG = [tmp retain]; 
[tmp release]; 

もう1つの潜在的な問題があります。あなたは が

[data getBytes:&tdata]; 
    if (tdata > 5) { 

ここ

getBytes:を

を非推奨メソッドを使用しているため、潜在的なバッファオーバーランの非推奨とGetBytesメソッドの結果は、 if(tdata > 5)のために使用することは適切ではありませんました。バイトの長さを確認したい場合は、 NSDataオブジェクトから取得します。

Answer 2

ここでは究極の問題であるかどうかは間違いです。最終的な条件文では、次のようになります。

cmessage=@""; 
... 
[cmessage release]; 

私はあなたがその文字列を解放する必要があるとは思わない。私は実際にstringWithFormat:またはinitWithString:とは対照的に、そのように定義されたNSStringのオブジェクトのインとアウトをすべて知っていませんが、その文字列オブジェクトの所有権はないと思います。あなたはそれを解放する必要はありません。

そのreleaseメッセージを削除し、役立つかどうか確認してください。