[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/khtml/html
From:       Maks Orlovich <maksim () kde ! org>
Date:       2010-11-27 17:28:50
Message-ID: 20101127172850.700CAAC8A2 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1201412 by orlovich:

Don't crash when someone (e.g. kmail) passes in some html w/o a KHTMLPart that contains 
a named iframe. It'd be best if HTMLQuoteColorer in kmail 4.4 were changed to use the 
part just in case, though. 

BUG: 257500


 M  +5 -0      html_baseimpl.cpp  


--- trunk/KDE/kdelibs/khtml/html/html_baseimpl.cpp #1201411:1201412
@@ -278,6 +278,11 @@
     // if the natural one conflicts
     KHTMLPart* parentPart = document()->part();
 
+    // If there is no part, we will not load anything, so no 
+    // worry about names being unique or not.
+    if (!parentPart)
+        return;
+
     KHTMLPart* otherFrame = parentPart->findFrame( name.string() );
     if (name.isEmpty() || (otherFrame && otherFrame != contentPart()))
         name = DOMString(parentPart->requestFrameName());
[prev in list] [next in list] [prev in thread] [next in thread]