Bellek Sızıntısı KeychainItemWrapper

Question

Ben kimlik doğrulaması için kullanıcı şifresini depolamak için Apple'ın örnek koddan KeyChainItemWrapper kullanıyorum ama şifreyi ayarlamak için çağırdığınızda:

[keychain setObject:passwordField.text forKey:(id)kSecValueData]; 

Bu gömleğimi tüm bellek sızıntıları geçiyor . bu KeyChainItemWrapper.m içinde 274 satıra geri izleri görünüşte sorun:

if (SecItemCopyMatching((CFDictionaryRef)genericPasswordQuery, (CFTypeRef *)&attributes) == noErr) 
{ 

nasıl düzeltebilirim ediyorum ve gelecekte Apple örnek kod ile çalışırken daha dikkatli olmalıdır?

Not: Daha fazla kod yayınlayabilirim, ancak bu satırları Aletleri kullanarak daralttım ve örnek kod tam olarak herhangi bir geliştiriciye açık.

Answer 1

KeyChainItemWrapper koduna bakarak, bu satırın bir bellek sızıntısı olduğunu kabul ediyorum. writeToKeychain'un sonunda [attributes release]'u kaçırdılar. Geri dönen referans nesnesini doğru şekilde nasıl serbest bıraktıklarına dair örnekler için bu dosyada SecItemCopyMatching() numaralı tüm çağrılara bakın.

Bu sayfanın altındaki "Bu iyi, ama ..." bağlantısını kullanıp hatayı not edin.

Answer 2

- (void) resetKeychainItem'da başka bir sızıntı buldum.

O

self.pKeychainItemData = [[[NSMutableDictionary alloc] init] autorelease]; olmalıdır.

Answer 3

Statik analiz, KeyChainItemWrapper.m adresindeki yöntem resetKeychainItem, satır 191'deki olası bir nesnenin sızıntısını bildiriyor. Şaşırtıcı bir şekilde, yukarıda belirtilen alanda potansiyel bir sızıntı olmadığını rapor etmekteyim, ancak önerilen nesnenin serbest bırakılmasını ve doğruluğunu ekledim. İşte

sızıntısı ile kod rapor ediliyor ise:

- (void)resetKeychainItem 
{ 
    ... 
    // Default attributes for keychain item. 
    [keychainItemData setObject:@"" forKey:(id)kSecAttrAccount]; // <-- Potential leak of an object 
    [keychainItemData setObject:@"" forKey:(id)kSecAttrLabel]; 
    [keychainItemData setObject:@"" forKey:(id)kSecAttrDescription]; 

    // Default data for keychain item. 
    [keychainItemData setObject:@"" forKey:(id)kSecValueData]; 
} 

Bu sorun "" @ boş dize üzerinde rapor ediliyor. Bu sorunu denemek ve düzeltmek için çeşitli kod uygulamalarını denedim, ancak hiçbir şey işe yaramıyor.

Bu yanlış pozitif midir?

Güncelleme: Göndermeden hemen sonra hatayı izlemek için uyarıyı çift tıklayabileceğimi farkettim.

Bu uyarı sözlük tahsis edilmesi için yukarıda hattına bağlanmaktadır:

if (!keychainItemData) 
{ 
    self.keychainItemData = [[[NSMutableDictionary alloc] init] autorelease]; 
} 

analiz uyarı artık mevcut değildir:

if (!keychainItemData) 
{ 
    self.keychainItemData = [[NSMutableDictionary alloc] init]; 
} 

aşağıdaki kod değiştirildi.