Loading...
libkern/c++/OSData.cpp xnu-4570.41.2 xnu-1504.9.26
--- xnu/xnu-4570.41.2/libkern/c++/OSData.cpp
+++ xnu/xnu-1504.9.26/libkern/c++/OSData.cpp
@@ -29,15 +29,11 @@
 
 #include <string.h>
 
-__BEGIN_DECLS
-#include <vm/vm_kern.h>
-__END_DECLS
-
 #include <libkern/c++/OSData.h>
 #include <libkern/c++/OSSerialize.h>
 #include <libkern/c++/OSLib.h>
 #include <libkern/c++/OSString.h>
-#include <IOKit/IOLib.h>
+#include <string.h>
 
 #define super OSObject
 
@@ -53,38 +49,32 @@
 
 #define EXTERNAL ((unsigned int) -1)
 
+#if OSALLOCDEBUG
+extern int debug_container_malloc_size;
+#define ACCUMSIZE(s) do { debug_container_malloc_size += (s); } while(0)
+#else
+#define ACCUMSIZE(s)
+#endif
+
 bool OSData::initWithCapacity(unsigned int inCapacity)
 {
-    if (data)
-    {
-        OSCONTAINER_ACCUMSIZE(-((size_t)capacity));
-	if (!inCapacity || (capacity < inCapacity))
-	{
-	    // clean out old data's storage if it isn't big enough
-	    if (capacity < page_size) kfree(data, capacity);
-	    else                      kmem_free(kernel_map, (vm_offset_t)data, capacity);
-	    data = 0;
-	    capacity = 0;
-	}
-    }
-
     if (!super::init())
         return false;
 
+    if (data && (!inCapacity || capacity < inCapacity) ) {
+        // clean out old data's storage if it isn't big enough
+        kfree(data, capacity);
+        data = 0;
+        ACCUMSIZE(-capacity);
+    }
+
     if (inCapacity && !data) {
-
-	if (inCapacity < page_size) data = (void *) kalloc_container(inCapacity);
-	else {
-	    kern_return_t kr;
-	    if (round_page_overflow(inCapacity, &inCapacity)) kr = KERN_RESOURCE_SHORTAGE;
-	    else kr = kmem_alloc(kernel_map, (vm_offset_t *)&data, inCapacity, IOMemoryTag(kernel_map));
-	    if (KERN_SUCCESS != kr) data = NULL;
-	}
+        data = (void *) kalloc(inCapacity);
         if (!data)
             return false;
         capacity = inCapacity;
-    }
-    OSCONTAINER_ACCUMSIZE(capacity);
+        ACCUMSIZE(inCapacity);
+    }
 
     length = 0;
     if (inCapacity < 16)
@@ -197,17 +187,15 @@
 
 void OSData::free()
 {
-    if ((capacity != EXTERNAL) && data && capacity) {
-	if (capacity < page_size) kfree(data, capacity);
-	else                      kmem_free(kernel_map, (vm_offset_t)data, capacity);
-        OSCONTAINER_ACCUMSIZE( -((size_t)capacity) );
+    if (capacity != EXTERNAL && data && capacity) {
+        kfree(data, capacity);
+        ACCUMSIZE( -capacity );
     } else if (capacity == EXTERNAL) {
-	DeallocFunction freemem = reserved ? reserved->deallocFunction : NULL;
-	if (freemem && data && length) {
-		freemem(data, length);
+	    DeallocFunction freemem = (DeallocFunction)reserved;
+		if (freemem && data && length) {
+			freemem(data, length);
+		}
 	}
-    }
-    if (reserved) kfree(reserved, sizeof(ExpansionData));
     super::free();
 }
 
@@ -229,51 +217,24 @@
 unsigned int OSData::ensureCapacity(unsigned int newCapacity)
 {
     unsigned char * newData;
-    unsigned int finalCapacity;
-    void * copydata;
-    kern_return_t kr;
 
     if (newCapacity <= capacity)
         return capacity;
 
-    finalCapacity = (((newCapacity - 1) / capacityIncrement) + 1)
+    newCapacity = (((newCapacity - 1) / capacityIncrement) + 1)
                 * capacityIncrement;
 
-    // integer overflow check
-    if (finalCapacity < newCapacity) return capacity;
-
-    copydata = data;
-
-    if (finalCapacity >= page_size) {
-	// round up
-	finalCapacity = round_page_32(finalCapacity);
-	// integer overflow check
-	if (finalCapacity < newCapacity) return capacity;
-	if (capacity >= page_size) {
-	    copydata = NULL;
-	    kr = kmem_realloc(kernel_map,
-			      (vm_offset_t)data,
-			      capacity,
-			      (vm_offset_t *)&newData,
-			      finalCapacity,
-			      IOMemoryTag(kernel_map));
-	} else {
-	    kr = kmem_alloc(kernel_map, (vm_offset_t *)&newData, finalCapacity, IOMemoryTag(kernel_map));
-	}
-	if (KERN_SUCCESS != kr) newData = NULL;
-    }
-    else newData = (unsigned char *) kalloc_container(finalCapacity);
-
+    newData = (unsigned char *) kalloc(newCapacity);
+    
     if ( newData ) {
-        bzero(newData + capacity, finalCapacity - capacity);
-        if (copydata) bcopy(copydata, newData, capacity);
+        bzero(newData + capacity, newCapacity - capacity);
         if (data) {
-	    if (capacity < page_size) kfree(data, capacity);
-	    else                      kmem_free(kernel_map, (vm_offset_t)data, capacity);
+            bcopy(data, newData, capacity);
+            kfree(data, capacity);
         }
-        OSCONTAINER_ACCUMSIZE( ((size_t)finalCapacity) - ((size_t)capacity) );
+        ACCUMSIZE( newCapacity - capacity );
         data = (void *) newData;
-        capacity = finalCapacity;
+        capacity = newCapacity;
     }
 
     return capacity;
@@ -343,7 +304,6 @@
 
     if (length
     &&  start < length
-    && (start + inLength) >= inLength // overflow check
     && (start + inLength) <= length)
         outData = (const void *) ((char *) data + start);
 
@@ -428,16 +388,12 @@
     unsigned int i;
     const unsigned char *p;
     unsigned char c;
-    unsigned int serializeLength;
 
     if (s->previouslySerialized(this)) return true;
 
     if (!s->addXMLStartTag(this, "data")) return false;
 
-    serializeLength = length;
-    if (reserved && reserved->disableSerialization) serializeLength = 0;
-
-    for (i = 0, p = (unsigned char *)data; i < serializeLength; i++, p++) {
+    for (i = 0, p = (unsigned char *)data; i < length; i++, p++) {
         /* 3 bytes are encoded as 4 */
         switch (i % 3) {
 	case 0:
@@ -475,29 +431,11 @@
     return s->addXMLEndTag("data");
 }
 
+/* Note I am just using the reserved pointer here instead of allocating a whole buffer
+ * to hold one pointer.
+ */
 void OSData::setDeallocFunction(DeallocFunction func)
 {
-    if (!reserved)
-    {
-    	reserved = (typeof(reserved)) kalloc_container(sizeof(ExpansionData));
-        if (!reserved) return;
-        bzero(reserved, sizeof(ExpansionData));
-    }
-    reserved->deallocFunction = func;
-}
-
-void OSData::setSerializable(bool serializable)
-{
-    if (!reserved)
-    {
-    	reserved = (typeof(reserved)) kalloc_container(sizeof(ExpansionData));
-	if (!reserved) return;
-	bzero(reserved, sizeof(ExpansionData));
-    }
-    reserved->disableSerialization = (!serializable);
-}
-
-bool OSData::isSerializable(void)
-{
-    return (!reserved || !reserved->disableSerialization);
-}
+    reserved = (ExpansionData *)func;
+	return;
+}