Changeset 54866 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

Feb 16, 2010, 8:06:48 PM (15 years ago)

Author:

[email protected]

Message:

Fix a handful of leaks seen on the buildbot.

Reviewed by Gavin Barraclough.

• runtime/UStringImpl.h:

(JSC::UStringOrRopeImpl::deref): Call URopeImpl::destructNonRecursive rather than delete
to ensure that the rope's fibers are also destroyed.

File:

• trunk/JavaScriptCore/runtime/UStringImpl.h (modified) (2 diffs)

trunk/JavaScriptCore/runtime/UStringImpl.h

@@ -341 +341 @@
     unsigned m_fiberCount;
     Fiber m_fibers[1];
+
+    friend class UStringOrRopeImpl;
 };
 
@@ -348 +350 @@
     if (!(m_refCountAndFlags & s_refCountMask)) {
         if (isRope())
-            delete static_cast<URopeImpl*>(this);
+            static_cast<URopeImpl*>(this)->destructNonRecursive();
         else if (!s_refCountFlagStatic)
             delete static_cast<UStringImpl*>(this);