Move the size check to be the first thing checked in CFAllocator functions

This saves us a lot of work is size is 0

Author: AZero13

CoreFoundation/Base.subproj/CFBase.c

@@ -581,8 +581,9 @@ CFAllocatorRef CFAllocatorCreate(CFAllocatorRef allocator, CFAllocatorContext *c
 
 void *CFAllocatorAllocate(CFAllocatorRef allocator, CFIndex size, CFOptionFlags hint) {
     CFAllocatorAllocateCallBack allocateFunc;
-    void *newptr = NULL;
+    void *newptr;
 
+    if (0 < size) {
     if (NULL == allocator) {
 	allocator = __CFGetDefaultAllocator();
     }
@@ -594,16 +595,14 @@ void *CFAllocatorAllocate(CFAllocatorRef allocator, CFIndex size, CFOptionFlags
 #else
     __CFGenericValidateType(allocator, _kCFRuntimeIDCFAllocator);
 #endif
-    if (0 == size) return NULL;
 #if TARGET_OS_MAC
     if (_CFTypeGetClass(allocator) != __CFISAForCFAllocator()) {	// malloc_zone_t *
 	return malloc_zone_malloc((malloc_zone_t *)allocator, size);
     }
 #endif
-    newptr = NULL;
     allocateFunc = __CFAllocatorGetAllocateFunction(&allocator->_context);
-    if (allocateFunc) {
-        newptr = (void *)INVOKE_CALLBACK3(allocateFunc, size, hint, allocator->_context.info);
+    if (NULL == allocateFunc) return NULL;
+    newptr = (void *)INVOKE_CALLBACK3(allocateFunc, size, hint, allocator->_context.info);
     }
     return newptr;
 }
@@ -631,14 +630,12 @@ void *CFAllocatorReallocate(CFAllocatorRef allocator, void *ptr, CFIndex newsize
 	    return malloc_zone_malloc((malloc_zone_t *)allocator, newsize);
 	}
 #endif
-	newptr = NULL;
 	allocateFunc = __CFAllocatorGetAllocateFunction(&allocator->_context);
-	if (allocateFunc) {
-		newptr = (void *)INVOKE_CALLBACK3(allocateFunc, newsize, hint, allocator->_context.info);
-	}
+	if (NULL == allocateFunc) return NULL;
+	newptr = (void *)INVOKE_CALLBACK3(allocateFunc, newsize, hint, allocator->_context.info);
 	return newptr;
     }
-    if (NULL != ptr && 0 == newsize) {
+    if (NULL != ptr && 0 >= newsize) {
 #if TARGET_OS_MAC
 	if (_CFTypeGetClass(allocator) != __CFISAForCFAllocator()) {	// malloc_zone_t *
 #if defined(DEBUG)
@@ -655,7 +652,7 @@ void *CFAllocatorReallocate(CFAllocatorRef allocator, void *ptr, CFIndex newsize
 	}
 	return NULL;
     }
-    if (NULL == ptr && 0 == newsize) return NULL;
+    if (NULL == ptr && 0 >= newsize) return NULL;
 #if TARGET_OS_MAC
     if (_CFTypeGetClass(allocator) != __CFISAForCFAllocator()) {	// malloc_zone_t *
 	return malloc_zone_realloc((malloc_zone_t *)allocator, ptr, newsize);
@@ -700,6 +697,7 @@ CFIndex CFAllocatorGetPreferredSizeForSize(CFAllocatorRef allocator, CFIndex siz
     CFAllocatorPreferredSizeCallBack prefFunc;
     CFIndex newsize = 0;
 
+    if (0 < size) {
     if (NULL == allocator) {
         allocator = __CFGetDefaultAllocator();
     }
@@ -717,10 +715,11 @@ CFIndex CFAllocatorGetPreferredSizeForSize(CFAllocatorRef allocator, CFIndex siz
     }
 #endif
     prefFunc = __CFAllocatorGetPreferredSizeFunction(&allocator->_context);
-    if (0 < size && NULL != prefFunc) {
+    if (NULL != prefFunc) {
 	newsize = (CFIndex)(INVOKE_CALLBACK3(prefFunc, size, hint, allocator->_context.info));
     }
     if (newsize < size) newsize = size;
+    }
     return newsize;
 }