Title: [289567] trunk/Source/WebCore
Revision
289567
Author
[email protected]
Date
2022-02-10 12:23:37 -0800 (Thu, 10 Feb 2022)

Log Message

Crash in in WebCore::CSSStyleSheet::didMutateRules
https://bugs.webkit.org/show_bug.cgi?id=236450

Patch by Gabriel Nava Marino <[email protected]> on 2022-02-10
Reviewed by Antti Koivisto.

Replace the raw pointer rule in RuleMutationScope with a RefPtr so it can be accessible
for the scope.

* css/CSSStyleSheet.cpp:
(WebCore::CSSStyleSheet::RuleMutationScope::~RuleMutationScope):
* css/CSSStyleSheet.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (289566 => 289567)


--- trunk/Source/WebCore/ChangeLog	2022-02-10 20:13:19 UTC (rev 289566)
+++ trunk/Source/WebCore/ChangeLog	2022-02-10 20:23:37 UTC (rev 289567)
@@ -1,3 +1,17 @@
+2022-02-10  Gabriel Nava Marino  <[email protected]>
+
+        Crash in in WebCore::CSSStyleSheet::didMutateRules
+        https://bugs.webkit.org/show_bug.cgi?id=236450
+
+        Reviewed by Antti Koivisto.
+
+        Replace the raw pointer rule in RuleMutationScope with a RefPtr so it can be accessible 
+        for the scope.
+
+        * css/CSSStyleSheet.cpp:
+        (WebCore::CSSStyleSheet::RuleMutationScope::~RuleMutationScope):
+        * css/CSSStyleSheet.h:
+
 2022-02-10  Tim Nguyen  <[email protected]>
 
         Unreviewed, update BundleResourceLoader WorkQueue identifier

Modified: trunk/Source/WebCore/css/CSSStyleSheet.cpp (289566 => 289567)


--- trunk/Source/WebCore/css/CSSStyleSheet.cpp	2022-02-10 20:13:19 UTC (rev 289566)
+++ trunk/Source/WebCore/css/CSSStyleSheet.cpp	2022-02-10 20:23:37 UTC (rev 289567)
@@ -418,7 +418,7 @@
 CSSStyleSheet::RuleMutationScope::~RuleMutationScope()
 {
     if (m_styleSheet)
-        m_styleSheet->didMutateRules(m_mutationType, m_contentsWereClonedForMutation, m_insertedKeyframesRule, m_modifiedKeyframesRuleName);
+        m_styleSheet->didMutateRules(m_mutationType, m_contentsWereClonedForMutation, m_insertedKeyframesRule.get(), m_modifiedKeyframesRuleName);
 }
 
 }

Modified: trunk/Source/WebCore/css/CSSStyleSheet.h (289566 => 289567)


--- trunk/Source/WebCore/css/CSSStyleSheet.h	2022-02-10 20:13:19 UTC (rev 289566)
+++ trunk/Source/WebCore/css/CSSStyleSheet.h	2022-02-10 20:23:37 UTC (rev 289567)
@@ -109,7 +109,7 @@
         CSSStyleSheet* m_styleSheet;
         RuleMutationType m_mutationType;
         WhetherContentsWereClonedForMutation m_contentsWereClonedForMutation;
-        StyleRuleKeyframes* m_insertedKeyframesRule;
+        RefPtr<StyleRuleKeyframes> m_insertedKeyframesRule;
         String m_modifiedKeyframesRuleName;
     };
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to