khtml still getting asserts on www.novinky.cz

Koos Vriezen koos.vriezen at xs4all.nl
Wed Dec 14 12:43:48 GMT 2005


On Wed, Dec 14, 2005 at 07:37:27AM +0100, Pavel Troller wrote:
> Index: html/html_imageimpl.cpp
> ===================================================================
> --- html/html_imageimpl.cpp	(revision 488116)
> +++ html/html_imageimpl.cpp	(working copy)
> @@ -224,6 +224,7 @@
>  
>  void HTMLImageElementImpl::attach()
>  {
> +    if (attached() || m_render) return; //SIN to prevent crash on novinky.cz
>      assert(!attached());
>      assert(!m_render);
>      assert(parentNode());

I wonder if it wrong that the impl classes assume correct tree
manipulations. Ie. shouldn't these errors be caught in a stage before like
the DOM or kjs classes?

Koos




More information about the kfm-devel mailing list