X-Git-Url: https://git.saurik.com/apple/security.git/blobdiff_plain/7fb2cbd2e68c73bf213d9907905431dbdb74c908..refs/heads/master:/OSX/libsecurity_keychain/lib/SecItem.cpp?ds=sidebyside

diff --git a/OSX/libsecurity_keychain/lib/SecItem.cpp b/OSX/libsecurity_keychain/lib/SecItem.cpp
index b1b571c2..e5b078ad 100644
--- a/OSX/libsecurity_keychain/lib/SecItem.cpp
+++ b/OSX/libsecurity_keychain/lib/SecItem.cpp
@@ -41,6 +41,8 @@
 #include "TrustSettingsSchema.h"
 #include <Security/SecTrustPriv.h>
 #include "utilities/array_size.h"
+#include "utilities/SecCFWrappers.h"
+#include "LegacyAPICounts.h"
 
 #include <AssertMacros.h>
 #include <syslog.h>
@@ -70,8 +72,6 @@ OSStatus SecItemAdd_ios(CFDictionaryRef attributes, CFTypeRef *result);
 OSStatus SecItemCopyMatching_ios(CFDictionaryRef query, CFTypeRef *result);
 OSStatus SecItemUpdate_ios(CFDictionaryRef query, CFDictionaryRef attributesToUpdate);
 OSStatus SecItemDelete_ios(CFDictionaryRef query);
-OSStatus SecItemUpdateTokenItems_ios(CFTypeRef tokenID, CFArrayRef tokenItemsAttributes);
-
 
 OSStatus SecItemValidateAppleApplicationGroupAccess(CFStringRef group);
 CFDictionaryRef SecItemCopyTranslatedAttributes(CFDictionaryRef inOSXDict, CFTypeRef itemClass,
@@ -3826,6 +3826,8 @@ AddItemResults(SecKeychainItemRef item,
 	//
 	// Note that we allocate *items if needed.
 
+	CFTypeRef localResult = NULL;
+
 	if (!item || !itemParams || !result)
 		return errSecParam;
 
@@ -3854,7 +3856,8 @@ AddItemResults(SecKeychainItemRef item,
 			CFArrayAppendValue(itemArray, itemRef);
 		}
 		else {
-			*result = CFRetain((CFTypeRef)itemRef);
+			CFReleaseNull(localResult);
+			localResult = CFRetain((CFTypeRef)itemRef);
 		}
 	}
 
@@ -3873,7 +3876,8 @@ AddItemResults(SecKeychainItemRef item,
 				CFArrayAppendValue(itemArray, persistentRef);
 			}
 			else {
-				*result = CFRetain(persistentRef);
+				CFReleaseNull(localResult);
+				localResult = CFRetain(persistentRef);
 			}
 			CFRelease(persistentRef);
 		}
@@ -3897,7 +3901,8 @@ AddItemResults(SecKeychainItemRef item,
 					CFArrayAppendValue(itemArray, dataRef);
 				}
 				else {
-					*result = CFRetain(dataRef);
+					CFReleaseNull(localResult);
+					localResult = CFRetain(dataRef);
 				}
 				CFRelease(dataRef);
 				status = errSecSuccess;
@@ -3919,7 +3924,8 @@ AddItemResults(SecKeychainItemRef item,
 					CFArrayAppendValue(itemArray, dataRef);
 				}
 				else {
-					*result = CFRetain(dataRef);
+					CFReleaseNull(localResult);
+					localResult = CFRetain(dataRef);
 				}
 				CFRelease(dataRef);
 				(void) SecKeychainItemFreeContent(NULL, data);
@@ -3948,7 +3954,8 @@ AddItemResults(SecKeychainItemRef item,
 				CFArrayAppendValue(itemArray, attrsDict);
 			}
 			else {
-				*result = CFRetain(attrsDict);
+				CFReleaseNull(localResult);
+				localResult = CFRetain(attrsDict);
 			}
 			CFRelease(attrsDict);
 		}
@@ -3961,14 +3968,22 @@ AddItemResults(SecKeychainItemRef item,
 		if (itemArray) {
 			CFArrayAppendValue(itemArray, itemDict);
 			CFRelease(itemDict);
-			*result = itemArray;
+			CFReleaseNull(localResult);
+			localResult = itemArray;
 		}
 		else {
-			*result = itemDict;
+			CFReleaseNull(localResult);
+			localResult = itemDict;
 		}
 	}
 	else if (itemArray) {
-		*result = itemArray;
+		CFReleaseNull(localResult);
+		localResult = itemArray;
+	}
+
+	if (localResult) {
+		*result = localResult;
+		localResult = NULL;
 	}
 
 	return status;
@@ -4309,7 +4324,7 @@ SecItemCreateFromAttributeDictionary_osx(CFDictionaryRef refAttributes) {
 		CFTypeRef v;
 
     Item item = Item(item_class, &attrs, 0, "");
-		v = CFDictionaryGetValue(refAttributes, kSecValuePersistentRef);
+		v = CFCast(CFData, CFDictionaryGetValue(refAttributes, kSecValuePersistentRef));
 		if (v) {
 			item->setPersistentRef((CFDataRef)v);
 		}
@@ -5081,14 +5096,6 @@ SecItemDelete(CFDictionaryRef query)
 	return status;
 }
 
-OSStatus
-SecItemUpdateTokenItems(CFTypeRef tokenID, CFArrayRef tokenItemsAttributes)
-{
-	OSStatus status = SecItemUpdateTokenItems_ios(tokenID, tokenItemsAttributes);
-	secitemlog(LOG_NOTICE, "SecItemUpdateTokenItems_ios result: %d", status);
-	return status;
-}
-
 OSStatus
 SecItemCopyMatching_osx(
 	CFDictionaryRef query,
@@ -5099,6 +5106,8 @@ SecItemCopyMatching_osx(
 	else
 		*result = NULL;
 
+    setCountLegacyAPIEnabledForThread(false);
+
 	CFAllocatorRef allocator = CFGetAllocator(query);
 	CFIndex matchCount = 0;
 	CFMutableArrayRef itemArray = NULL;
@@ -5143,20 +5152,9 @@ error_exit:
 	}
 	_FreeSecItemParams(itemParams);
 
-	return status;
-}
+    setCountLegacyAPIEnabledForThread(true);
 
-OSStatus
-SecItemCopyDisplayNames(
-	CFArrayRef items,
-	CFArrayRef *displayNames)
-{
-    BEGIN_SECAPI
-	Required(items);
-	Required(displayNames);
-    //%%%TBI
-    return errSecUnimplemented;
-    END_SECAPI
+	return status;
 }
 
 OSStatus
@@ -5169,6 +5167,8 @@ SecItemAdd_osx(
 	else if (result)
 		*result = NULL;
 
+    setCountLegacyAPIEnabledForThread(false);
+
 	CFAllocatorRef allocator = CFGetAllocator(attributes);
 	CFMutableArrayRef itemArray = NULL;
 	SecKeychainItemRef item = NULL;
@@ -5302,6 +5302,7 @@ error_exit:
 		*result = NULL;
 	}
 	_FreeSecItemParams(itemParams);
+    setCountLegacyAPIEnabledForThread(true);
 
 	return status;
 }
@@ -5329,6 +5330,8 @@ SecItemUpdate_osx(
 		CFRelease(results);
 	}
 
+    setCountLegacyAPIEnabledForThread(false);
+
 	OSStatus result = errSecSuccess;
 	CFIndex ix, count = CFArrayGetCount(items);
 	for (ix=0; ix < count; ix++) {
@@ -5339,6 +5342,8 @@ SecItemUpdate_osx(
 		}
 	}
 
+    setCountLegacyAPIEnabledForThread(true);
+
 	if (items) {
 		CFRelease(items);
 	}
@@ -5367,6 +5372,8 @@ SecItemDelete_osx(
 		CFRelease(results);
 	}
 
+    setCountLegacyAPIEnabledForThread(false);
+
 	OSStatus result = errSecSuccess;
 	CFIndex ix, count = CFArrayGetCount(items);
 	for (ix=0; ix < count; ix++) {
@@ -5382,6 +5389,8 @@ SecItemDelete_osx(
 		}
 	}
 
+    setCountLegacyAPIEnabledForThread(true);
+
 	if (items)
 		CFRelease(items);