Changeset 79594 in webkit for trunk/Source/JavaScriptCore/yarr/YarrPattern.cpp

Timestamp:

Feb 24, 2011, 11:25:27 AM (14 years ago)

Author:

[email protected]

Message:

2011-02-24 Michael Saboff <​[email protected]>

Reviewed by Oliver Hunt.

PatternAlternative leaked in YarrPatternConstructor::atomParenthesesEnd()
​https://bugs.webkit.org/show_bug.cgi?id=55156

Added code to delete unneeded PatternAlternative after it is removed
from m_alternatives Vector.

• yarr/YarrPattern.cpp: (JSC::Yarr::YarrPatternConstructor::atomParenthesesEnd):

File:

• trunk/Source/JavaScriptCore/yarr/YarrPattern.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/yarr/YarrPattern.cpp

@@ -496 +496 @@
         for (unsigned i = 0; i < numParenAlternatives; i++) {
             if (!parenthesesDisjunction->m_alternatives[i]->m_terms.size() && numParenAlternatives > 1) {
+                PatternAlternative* altToRemove = parenthesesDisjunction->m_alternatives[i];
                 parenthesesDisjunction->m_alternatives.remove(i);
+                delete altToRemove;
                 --numParenAlternatives;