Changeset 38673 in webkit for trunk/JavaScriptCore/wtf/FastMalloc.cpp

Timestamp:

Nov 21, 2008, 1:20:41 PM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=22402
Replace abort() with CRASH()

• wtf/Assertions.h: Added a different method to crash, which should work even is 0xbbadbeef is a valid memory address.

• runtime/Collector.cpp:
• wtf/FastMalloc.cpp:
• wtf/FastMalloc.h:
• wtf/TCSpinLock.h: Replace abort() with CRASH().

File:

• trunk/JavaScriptCore/wtf/FastMalloc.cpp (modified) (15 diffs)

trunk/JavaScriptCore/wtf/FastMalloc.cpp

@@ -192 +192 @@
     void* result = malloc(n);
     if (!result)
-        abort();
+        CRASH();
     return result;
 }
@@ -207 +207 @@
     void* result = calloc(n_elements, element_size);
     if (!result)
-        abort();
+        CRASH();
     return result;
 }
@@ -228 +228 @@
     void* result = realloc(p, n);
     if (!result)
-        abort();
+        CRASH();
     return result;
 }
@@ -687 +687 @@
   if (ClassIndex(0) < 0) {
     MESSAGE("Invalid class index %d for size 0\n", ClassIndex(0));
-    abort();
+    CRASH();
   }
   if (static_cast<size_t>(ClassIndex(kMaxSize)) >= sizeof(class_array)) {
     MESSAGE("Invalid class index %d for kMaxSize\n", ClassIndex(kMaxSize));
-    abort();
+    CRASH();
   }
 
@@ -743 +743 @@
     MESSAGE("wrong number of size classes: found %" PRIuS " instead of %d\n",
             sc, int(kNumClasses));
-    abort();
+    CRASH();
   }
 
@@ -761 +761 @@
     if (sc == 0) {
       MESSAGE("Bad size class %" PRIuS " for %" PRIuS "\n", sc, size);
-      abort();
+      CRASH();
     }
     if (sc > 1 && size <= class_to_size[sc-1]) {
       MESSAGE("Allocating unnecessarily large class %" PRIuS " for %" PRIuS
               "\n", sc, size);
-      abort();
+      CRASH();
     }
     if (sc >= kNumClasses) {
       MESSAGE("Bad size class %" PRIuS " for %" PRIuS "\n", sc, size);
-      abort();
+      CRASH();
     }
     const size_t s = class_to_size[sc];
     if (size > s) {
      MESSAGE("Bad size %" PRIuS " for %" PRIuS " (sc = %" PRIuS ")\n", s, size, sc);
-      abort();
+      CRASH();
     }
     if (s == 0) {
       MESSAGE("Bad size %" PRIuS " for %" PRIuS " (sc = %" PRIuS ")\n", s, size, sc);
-      abort();
+      CRASH();
     }
   }
@@ -862 +862 @@
         // Need more room
         free_area_ = reinterpret_cast<char*>(MetaDataAlloc(kAllocIncrement));
-        if (free_area_ == NULL) abort();
+        if (free_area_ == NULL) CRASH();
         free_avail_ = kAllocIncrement;
       }
@@ -3024 +3024 @@
 
 #ifdef WTF_CHANGES
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 #endif
 static ALWAYS_INLINE void* do_malloc(size_t size) {
@@ -3057 +3057 @@
   if (!ret) {
 #ifdef WTF_CHANGES
-    if (abortOnFailure) // This branch should be optimized out by the compiler.
-        abort();
+    if (crashOnFailure) // This branch should be optimized out by the compiler.
+        CRASH();
 #else
     errno = ENOMEM;
@@ -3227 +3227 @@
 extern "C" 
 #else
-#define do_malloc do_malloc<abortOnFailure>
-
-template <bool abortOnFailure>
+#define do_malloc do_malloc<crashOnFailure>
+
+template <bool crashOnFailure>
 void* malloc(size_t);
 
@@ -3242 +3242 @@
 }
 
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 ALWAYS_INLINE
 #endif
@@ -3266 +3266 @@
 extern "C" 
 #else
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 void* calloc(size_t, size_t);
 
@@ -3279 +3279 @@
 }
 
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 ALWAYS_INLINE
 #endif
@@ -3312 +3312 @@
 extern "C" 
 #else
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 void* realloc(void*, size_t);
 
@@ -3325 +3325 @@
 }
 
-template <bool abortOnFailure>
+template <bool crashOnFailure>
 ALWAYS_INLINE
 #endif