Loading...
libkern/c++/OSString.cpp xnu-3248.50.21 xnu-2782.40.9
--- xnu/xnu-3248.50.21/libkern/c++/OSString.cpp
+++ xnu/xnu-2782.40.9/libkern/c++/OSString.cpp
@@ -56,6 +56,15 @@
 OSMetaClassDefineReservedUnused(OSString, 14);
 OSMetaClassDefineReservedUnused(OSString, 15);
 
+#if OSALLOCDEBUG
+extern "C" {
+    extern int debug_container_malloc_size;
+};
+#define ACCUMSIZE(s) do { debug_container_malloc_size += (s); } while(0)
+#else
+#define ACCUMSIZE(s)
+#endif
+
 bool OSString::initWithString(const OSString *aString)
 {
     return initWithCString(aString->string);
@@ -63,54 +72,35 @@
 
 bool OSString::initWithCString(const char *cString)
 {
-    unsigned int   newLength;
-    char         * newString;
-
-    if (!cString || !super::init()) return false;
-
-    newLength = strlen(cString) + 1;
-    newString = (char *) kalloc_container(newLength);
-    if (!newString) return false;
-
-    bcopy(cString, newString, newLength);
-
-    if ( !(flags & kOSStringNoCopy) && string) {
-        kfree(string, (vm_size_t)length);
-        OSCONTAINER_ACCUMSIZE(-((size_t)length));
-    }
-    string = newString;
-    length = newLength;
-    flags &= ~kOSStringNoCopy;
-
-    OSCONTAINER_ACCUMSIZE(length);
+    if (!cString || !super::init())
+        return false;
+
+    length = strlen(cString) + 1;
+    string = (char *) kalloc(length);
+    if (!string)
+        return false;
+
+    bcopy(cString, string, length);
+
+    ACCUMSIZE(length);
 
     return true;
 }
 
 bool OSString::initWithStringOfLength(const char *cString, size_t inlength)
 {
-    unsigned int   newLength;
-    char         * newString;
-
-    if (!cString || !super::init()) return false;
-
-    newLength = inlength + 1;
-    newString = (char *) kalloc_container(newLength);
-    if (!newString) return false;
-
-    bcopy(cString, newString, inlength);
-    newString[inlength] = 0;
-
-    if ( !(flags & kOSStringNoCopy) && string) {
-        kfree(string, (vm_size_t)length);
-        OSCONTAINER_ACCUMSIZE(-((size_t)length));
-    }
-
-    string = newString;
-    length = newLength;
-    flags &= ~kOSStringNoCopy;
-
-    OSCONTAINER_ACCUMSIZE(length);
+    if (!cString || !super::init())
+        return false;
+
+    length = inlength + 1;
+    string = (char *) kalloc(length);
+    if (!string)
+        return false;
+
+    bcopy(cString, string, inlength);
+    string[inlength] = 0;
+
+    ACCUMSIZE(length);
 
     return true;
 }
@@ -207,7 +197,7 @@
 {
     if ( !(flags & kOSStringNoCopy) && string) {
         kfree(string, (vm_size_t)length);
-        OSCONTAINER_ACCUMSIZE(-((size_t)length));
+        ACCUMSIZE(-length);
     }
 
     super::free();