Reviewed by John.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Dec 2003 01:09:01 +0000 (01:09 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Dec 2003 01:09:01 +0000 (01:09 +0000)
<rdar://problem/3488220>: namespaces test from David Faure causes Safari to crash in DOM::DocumentImpl::createElementNS

        * khtml/xml/dom_docimpl.cpp:
        (DocumentImpl::createElementNS): Check if new element is NULL
before messing with it, to avoid a crash.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/xml/dom_docimpl.cpp

index f8ce51d..617e06d 100644 (file)
@@ -1,3 +1,13 @@
+2003-12-02  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by John.
+
+       <rdar://problem/3488220>: namespaces test from David Faure causes Safari to crash in DOM::DocumentImpl::createElementNS
+       
+        * khtml/xml/dom_docimpl.cpp:
+        (DocumentImpl::createElementNS): Check if new element is NULL
+       before messing with it, to avoid a crash.
+
 2003-12-02  David Hyatt  <hyatt@apple.com>
 
        Fix for 3497999, null check the text() of RenderTexts to avoid a crash in current().
index 6b416ae..17b821d 100644 (file)
@@ -516,7 +516,7 @@ ElementImpl *DocumentImpl::createElementNS( const DOMString &_namespaceURI, cons
         // (elements not in this namespace are treated as normal XML elements)
         e = createHTMLElement(qName.mid(colonPos+1));
         int exceptioncode = 0;
-        if (colonPos >= 0)
+        if (e && colonPos >= 0)
             e->setPrefix(qName.left(colonPos),  exceptioncode);
     }
     if (!e)