Memory leak in KIconLoader

Jakob Petsovits jpetso at gmx.at
Tue May 27 07:37:06 BST 2008


On Monday, 12. May 2008, Jakob Petsovits wrote:
> On Sunday, 11. May 2008, Jakob Petsovits wrote:
> > On Saturday, 10. May 2008, Michael Jansen wrote:
> > > That code was introduced with change 794709. The nodes will be delete
> > > with a qDeleteAll(links) so some nodes are leaked. I'm not sure how to
> > > fix it so i put it here.
> >
> > This patch should do the trick, right?
>
> Arr, crap, the if condition goes exactly the wrong way - thanks to Urs
> Wolfer for noticing. Here's the corrected patch (still untested).

Now tested, does not seem to break everything in horrible ways.
I committed that patch in r813190, speak up if you notice any shortcomings.

Wishes,
  Jakob




More information about the kde-core-devel mailing list