SecItemCopyMatching memory leaking

Question

Ho una perdita di memoria nel codice successivo. Mi sono ispirato allo here e questa è una parte dell'algoritmo RSA.

- (SecKeyRef)getPublicKeyRef { 
OSStatus resultCode = noErr; 
SecKeyRef publicKeyReference = NULL; 

if(publicKey == NULL) { 
    NSMutableDictionary * queryPublicKey = [[NSMutableDictionary alloc] init]; 

    NSData *publicTag = [NSData dataWithBytes:publicKeyIdentifier 

             length:strlen((const char *)publicKeyIdentifier)]; 

    // Set the public key query dictionary. 
    [queryPublicKey setObject:(id)kSecClassKey forKey:(id)kSecClass]; 
    [queryPublicKey setObject:publicTag forKey:(id)kSecAttrApplicationTag]; 

    [queryPublicKey setObject:(id)kSecAttrKeyTypeRSA forKey:(id)kSecAttrKeyType]; 

    [queryPublicKey setObject:[NSNumber numberWithBool:YES] forKey:(id)kSecReturnRef]; 

    // Get the key.  
    resultCode = SecItemCopyMatching((CFDictionaryRef)queryPublicKey, (CFTypeRef *)&publicKeyReference); 
    // NSLog(@"getPublicKey: result code: %d", resultCode); 

    if(resultCode != noErr) 
    { 
     publicKeyReference = NULL; 
    } 

    // [publicTag release]; 
    [queryPublicKey release]; 
} else { 
    publicKeyReference = publicKey; 
} 

return publicKeyReference; 

}

Lo strumento Leak dice che sta perdendo in questa linea:

resultCode = SecItemCopyMatching((CFDictionaryRef)queryPublicKey, (CFTypeRef *)&publicKeyReference); 

dirmi come avrei potuto risolvere questo problema.

Answer 1

Il tuo metodo è a volte restituisce un'istanza con il conteggio dei ritegni +1 e probabilmente non la rilascerai nel resto del codice. Stai ritornando con retain count con +1 se viene chiamato SecItemCopyMatching, ma se publicKey è impostato, la tua funzione restituisce un valore con retain count + -0 che è negativo.

È necessario assicurarsi di restituire sempre con lo stesso conteggio. In questo caso, mi piacerebbe fare:

} else { 
    publicKeyReference = publicKey; 
    CFRetain(publicKeyReference); 
} 

Poi ogni visitatore al vostro metodo deve fare in modo di CFRelease il valore ... ma che sarebbe violare la get regola (deve restituire mantenere count + - 0), quindi forse sarebbe una buona idea rinominare il metodo.