Fixed memory leak in SecureDataStore

DoubleSha · Oct 27, 2015 · 4db5690 · 4db5690
1 parent d4f76e2
commit 4db5690
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/BitcoinSwift/OSKeyChainSecureDataStore.swift b/BitcoinSwift/OSKeyChainSecureDataStore.swift
@@ -25,18 +25,16 @@ public class OSKeyChainSecureDataStore: SecureDataStore {
     query.setObject(service, forKey: "\(kSecAttrService)")
     query.setObject(key, forKey: "\(kSecAttrAccount)")
     query.setObject(true, forKey: "\(kSecReturnData)")
-    let result = UnsafeMutablePointer<AnyObject?>.alloc(1)
-    result.initialize(nil)
-    let status = SecItemCopyMatching(query, result)
+    var result: CFTypeRef? = nil
+    let status = SecItemCopyMatching(query, &result)
     if status == errSecItemNotFound {
       return nil
     }
     if status != noErr {
       return nil
     }
-    let data = result.memory as? NSData
+    let data = result as! NSData
     let secureData = SecureData(data: data)
-    result.dealloc(1)
     return secureData
   }