Memory leak in KIconLoader

Jakob Petsovits jpetso at gmx.at
Mon May 12 11:03:36 BST 2008


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).

Wishes,
  Jakob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: iconloader-fix-memory-leak.diff
Type: text/x-diff
Size: 807 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080512/08a823c6/attachment.diff>


More information about the kde-core-devel mailing list