Crash on http://news.google.com

CP Hennessy CP.Hennessy at iname.com
Mon Jun 2 16:53:45 BST 2003


Hi again Dirk,
	OK, this seems like it does not crash konqi on news.google.com.
So you patch worked. You mentioned that it may add another race, 
so I'll keep using it with the patch to see if there are any problems 
with it.

Thanks
CPH

On Monday 02 June 2003 16:30, CP Hennessy wrote:
> Hi Dirk,
>   This was Qt 3.1.
> However I did not see any response from anyone, so yesterday
> I did a cvs update to get the latest qt-coy, arts, kdelibs, and kdebase.
>  I've just opened this newly compiled konqueror and have what I think
> is the same problem!
>
> I'll now try your patch and give you an update shortly.
>
> CPH
>
> On Monday 02 June 2003 13:11, Dirk Mueller wrote:
> > On Mon, 02 Jun 2003, Dirk Mueller wrote:
> > > The standard solution to this problem would be:
> >
> > I just noticed that QMove is not a QObject. try this patch instead:
> >
> > --- loader.cpp  5 May 2003 17:26:06 -0000       1.145
> > +++ loader.cpp  2 Jun 2003 12:10:44 -0000
> > @@ -805,11 +805,12 @@ void CachedImage::deleteMovie()
> >      delete m; m = 0;
> >  }
> >
> >  void CachedImage::clear()
> >  {
> > -    delete m;   m = 0;
> > +    if ( m )
> > +        QTimer::singleShot(0, this, SLOT( deleteMovie()));
> >      delete p;   p = 0;
> >      delete bg;  bg = 0;
> >      bgColor = qRgba( 0, 0, 0, 0xff );
> >      delete pixPart; pixPart = 0;
> >
> > of course this opens a tiny window for new races.. :-(




More information about the kfm-devel mailing list