Title: [155195] trunk/Source/WebCore
Revision
155195
Author
[email protected]
Date
2013-09-06 10:33:44 -0700 (Fri, 06 Sep 2013)

Log Message

Fix fastMalloc/delete mismatches in CSS parser
https://bugs.webkit.org/show_bug.cgi?id=120864

Reviewed by Andreas Kling.

Stop putting fastMalloced memory inside an OwnPtr.

* css/CSSParserValues.cpp:
(WebCore::CSSParserSelector::CSSParserSelector):
(WebCore::CSSParserSelector::adoptSelectorVector):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (155194 => 155195)


--- trunk/Source/WebCore/ChangeLog	2013-09-06 17:23:03 UTC (rev 155194)
+++ trunk/Source/WebCore/ChangeLog	2013-09-06 17:33:44 UTC (rev 155195)
@@ -1,3 +1,16 @@
+2013-09-06  Anders Carlsson  <[email protected]>
+
+        Fix fastMalloc/delete mismatches in CSS parser
+        https://bugs.webkit.org/show_bug.cgi?id=120864
+
+        Reviewed by Andreas Kling.
+
+        Stop putting fastMalloced memory inside an OwnPtr.
+
+        * css/CSSParserValues.cpp:
+        (WebCore::CSSParserSelector::CSSParserSelector):
+        (WebCore::CSSParserSelector::adoptSelectorVector):
+
 2013-09-06  Andreas Kling  <[email protected]>
 
         FrameLoader::subresourceLoader() should return a reference.

Modified: trunk/Source/WebCore/css/CSSParserValues.cpp (155194 => 155195)


--- trunk/Source/WebCore/css/CSSParserValues.cpp	2013-09-06 17:23:03 UTC (rev 155194)
+++ trunk/Source/WebCore/css/CSSParserValues.cpp	2013-09-06 17:33:44 UTC (rev 155195)
@@ -149,7 +149,7 @@
 }
 
 CSSParserSelector::CSSParserSelector()
-    : m_selector(adoptPtr(fastNew<CSSSelector>()))
+    : m_selector(adoptPtr(new CSSSelector))
 {
 }
 
@@ -175,9 +175,9 @@
 
 void CSSParserSelector::adoptSelectorVector(Vector<OwnPtr<CSSParserSelector> >& selectorVector)
 {
-    CSSSelectorList* selectorList = fastNew<CSSSelectorList>();
+    OwnPtr<CSSSelectorList> selectorList = adoptPtr(new CSSSelectorList);
     selectorList->adoptSelectorVector(selectorVector);
-    m_selector->setSelectorList(adoptPtr(selectorList));
+    m_selector->setSelectorList(selectorList.release());
 }
 
 bool CSSParserSelector::isSimple() const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to