Title: [98796] trunk/Source/WebCore
Revision
98796
Author
[email protected]
Date
2011-10-28 23:31:16 -0700 (Fri, 28 Oct 2011)

Log Message

Potential crash in ReplaceNodeWithSpanCommand
https://bugs.webkit.org/show_bug.cgi?id=71145

Reviewed by Ojan Vafai.

Fix a potential crash without tests because we don't have a reduction.

* editing/ReplaceNodeWithSpanCommand.cpp:
(WebCore::swapInNodePreservingAttributesAndChildren):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (98795 => 98796)


--- trunk/Source/WebCore/ChangeLog	2011-10-29 05:51:51 UTC (rev 98795)
+++ trunk/Source/WebCore/ChangeLog	2011-10-29 06:31:16 UTC (rev 98796)
@@ -1,3 +1,15 @@
+2011-10-28  Ryosuke Niwa  <[email protected]>
+
+        Potential crash in ReplaceNodeWithSpanCommand
+        https://bugs.webkit.org/show_bug.cgi?id=71145
+
+        Reviewed by Ojan Vafai.
+
+        Fix a potential crash without tests because we don't have a reduction.
+
+        * editing/ReplaceNodeWithSpanCommand.cpp:
+        (WebCore::swapInNodePreservingAttributesAndChildren):
+
 2011-10-28  Adam Barth  <[email protected]>
 
         MessagePort should be a ContextDestructionObserver

Modified: trunk/Source/WebCore/editing/ReplaceNodeWithSpanCommand.cpp (98795 => 98796)


--- trunk/Source/WebCore/editing/ReplaceNodeWithSpanCommand.cpp	2011-10-29 05:51:51 UTC (rev 98795)
+++ trunk/Source/WebCore/editing/ReplaceNodeWithSpanCommand.cpp	2011-10-29 06:31:16 UTC (rev 98796)
@@ -52,7 +52,7 @@
 {
     ASSERT(nodeToReplace->inDocument());
     ExceptionCode ec = 0;
-    ContainerNode* parentNode = nodeToReplace->parentNode();
+    RefPtr<ContainerNode> parentNode = nodeToReplace->parentNode();
     parentNode->insertBefore(newNode, nodeToReplace, ec);
     ASSERT(!ec);
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to