Changeset 206069 in webkit

Timestamp:

Sep 17, 2016 3:48:23 AM (8 years ago)

Author:

Carlos Garcia Campos

Message:

Merge r205998 - REGRESSION (r205462): Lot of leaks
​https://bugs.webkit.org/show_bug.cgi?id=161946

Reviewed by Saam Barati.

We were forgetting to delete LargeAllocations on VM exit!

• heap/MarkedSpace.cpp:

(JSC::MarkedSpace::~MarkedSpace):

Location:

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• heap/MarkedSpace.cpp (modified) (1 diff)

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2016-09-15  Filip Pizlo  <fpizlo@apple.com>
+
+        REGRESSION (r205462): Lot of leaks
+        https://bugs.webkit.org/show_bug.cgi?id=161946
+
+        Reviewed by Saam Barati.
+        
+        We were forgetting to delete LargeAllocations on VM exit!
+
+        * heap/MarkedSpace.cpp:
+        (JSC::MarkedSpace::~MarkedSpace):
+
 2016-09-06  Saam Barati  <sbarati@apple.com>
 

releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/heap/MarkedSpace.cpp

@@ -204 +204 @@
             freeBlock(block);
         });
+    for (LargeAllocation* allocation : m_largeAllocations)
+        allocation->destroy();
     ASSERT(!m_blocks.set().size());
 }