In CrMallocErrorBreak, do not kill the process if errno is ENOMEM.

base/process_util_mac.mm

Index: base/process_util_mac.mm
diff --git a/base/process_util_mac.mm b/base/process_util_mac.mm
index 6b39a614e4404d81991f0694159ed76e89e86d8b..39edb29d3e34c5c2f27b87a72b9a8646d4c50f34 100644
--- a/base/process_util_mac.mm
+++ b/base/process_util_mac.mm
@@ -549,6 +549,13 @@ malloc_error_break_t LookUpMallocErrorBreak() {
 
 void CrMallocErrorBreak() {
   g_original_malloc_error_break();
+
+  // Out of memory is certainly not heap corruption, and not necessairly

[Scott Hess - ex-Googler, 2012/03/05 20:34:20]

s/necessairly/correct spelling/

Also, I think the OOM killer uniformly decides to crash in case of NULL.

[Robert Sesek, 2012/03/06 01:16:27]

Spalling es ovarraited.

+  // something for which the process should be terminated. Leave that decision
+  // to the OOM killer.
+  if (errno == ENOMEM)
+    return;
+
   // A unit test checks this error message, so it needs to be in release builds.
   LOG(ERROR) <<
       "Terminating process due to a potential for future heap corruption";