unity-dev team mailing list archive

Thread
Date

Re: [Ayatana-dev] Patch to fix memory leak in WindowsList::removeWindow()

To: Richard Dale <richard.dale@xxxxxxxxxxxxxxx>
Subject: Re: [Ayatana-dev] Patch to fix memory leak in WindowsList::removeWindow()
From: Aurélien Gâteau <aurelien.gateau@xxxxxxxxxxxxx>
Date: Tue, 12 Apr 2011 18:28:34 +0200
Cc: ayatana-dev@xxxxxxxxxxxxxxxxxxx
In-reply-to: <201104121524.26936.richard.dale@codethink.co.uk>
List-archive: <http://lists.launchpad.net/ayatana-dev>
List-help: <https://help.launchpad.net/ListHelp>
List-id: <ayatana-dev.lists.launchpad.net>
List-owner: <https://launchpad.net/~ayatana-dev>
List-post: <mailto:ayatana-dev@lists.launchpad.net>
List-subscribe: <https://launchpad.net/~ayatana-dev>
List-unsubscribe: <https://launchpad.net/~ayatana-dev>
References: <201104121524.26936.richard.dale@codethink.co.uk>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110406 Thunderbird/3.1.9

On 12/04/2011 16:24, Richard Dale wrote:

While I was investigating a unity-2d-spread crash bug I noticed that there was
a memory leak in the WindowsList::removeWindow() method. When a WindowInfo*
pointer was removed from the list it wasn't being deleted. Please find attached
a patch to fix this.


Good catch, applied.

Aurélien

References

[Ayatana-dev] Patch to fix memory leak in WindowsList::removeWindow()
From: Richard Dale, 2011-04-12