After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 428072 - leaks bonobo ui nodes
leaks bonobo ui nodes
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: keyboard indicator (gswitchit)
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-04-09 23:57 UTC by Matthias Clasen
Modified: 2008-11-10 00:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
node-leak.patch (466 bytes, patch)
2007-04-09 23:58 UTC, Matthias Clasen
accepted-commit_now Details | Review

Description Matthias Clasen 2007-04-09 23:57:30 UTC
Since it doesn't unref them after use. Trivial patch below.
Comment 1 Matthias Clasen 2007-04-09 23:58:19 UTC
Created attachment 86083 [details] [review]
node-leak.patch
Comment 2 Cosimo Cecchi 2008-03-03 11:22:50 UTC
This still applies, can someone please review it?
Comment 3 Matthias Clasen 2008-11-09 18:28:47 UTC
Ping. Any objection to committing this simple fix ?
Comment 4 Callum McKenzie 2008-11-09 19:40:30 UTC
Go for it.
Comment 5 Cosimo Cecchi 2008-11-09 23:42:21 UTC
Setting patch status.
Comment 6 Matthias Clasen 2008-11-10 00:04:39 UTC
2008-11-09  Matthias Clasen  <mclasen@redhat.com>

        Bug 428072 – leaks bonobo ui nodes

        * gswitchit-applet.c (GSwitchItAppletSetupGroupsSubmenu): Don't
        leak the BonoboUINode.
Comment 7 Sergey V. Udaltsov 2008-11-10 00:13:45 UTC
Another lost bug:(. Thanks for committing!