Suggested by Darin.

        * khtml/html/HTMLElementImpl.cpp:
        (WebCore::HTMLElementImpl::createContextualFragment):
        - make nextChild a RefPtr to survive possible DOM changes

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@12604 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 95b2ff621bd66f5eba1f59b219ffe0fede8b259e..081597363dace7beb8f7308a6df45355ada356a9 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,11 @@
+2006-02-06  David Harrison  <harrison@apple.com>
+
+        Suggested by Darin.
+        
+        * khtml/html/HTMLElementImpl.cpp:
+        (WebCore::HTMLElementImpl::createContextualFragment):
+        - make nextChild a RefPtr to survive possible DOM changes
+
 2006-02-06  Beth Dakin  <bdakin@apple.com>
 
         Reviewed by John and Adele

diff --git a/WebCore/khtml/html/HTMLElementImpl.cpp b/WebCore/khtml/html/HTMLElementImpl.cpp
index 1bbcf34265fb79cbdbd149f74a74022525fc4ae8..ecc9f9b39ab05ce47ffaeaf1ca2db022bd9569b1 100644 (file)
--- a/WebCore/khtml/html/HTMLElementImpl.cpp
+++ b/WebCore/khtml/html/HTMLElementImpl.cpp
@@ -274,7 +274,7 @@ DocumentFragmentImpl *HTMLElementImpl::createContextualFragment(const DOMString
             NodeImpl *firstChild = node->firstChild();
             if (firstChild)
                 nextNode = firstChild;
-            NodeImpl *nextChild;
+            RefPtr<NodeImpl> nextChild;
             for (RefPtr<NodeImpl> child = firstChild; child; child = nextChild) {
                 nextChild = child->nextSibling();
                 node->removeChild(child.get(), ignoredExceptionCode);