2010-12-15 14:33:31 |
Chow Loong Jin |
attachment added |
|
dbusmenu-memleak-fix.patch https://bugs.launchpad.net/bugs/690668/+attachment/1766779/+files/dbusmenu-memleak-fix.patch |
|
2010-12-15 14:33:31 |
Chow Loong Jin |
attachment added |
|
indicator-messages-memleak-fix.patch https://bugs.launchpad.net/bugs/690668/+attachment/1766780/+files/indicator-messages-memleak-fix.patch |
|
2010-12-15 14:33:31 |
Chow Loong Jin |
attachment added |
|
libindicate-memleak-fix.patch https://bugs.launchpad.net/bugs/690668/+attachment/1766781/+files/libindicate-memleak-fix.patch |
|
2010-12-15 14:33:31 |
Chow Loong Jin |
bug |
|
|
added bug |
2010-12-15 14:36:10 |
Chow Loong Jin |
description |
affects libindicate
affects dbusmenu
affects indicator-messages
As per the title, these projects leak memory. dbus_g_proxy_end_call requires you
to free the memory it allocates, including the GErrors. The patch attached does
that.
indicator-messages has an additional leak, which I'm not sure is libdbusmenu's
or indicator-messages' fault. libdbusmenu passes a GdkPixbuf into
indicator-messages, indicator-messages scales it and doesn't free the original
pixbuf. However, libdbusmenu doesn't free those pixbufs either, so it gets
stored. The same thing happened with indicator-sound, but the patch for that has
already been committed.
These patches are by no means complete. I'm just hunting down memleaks one by
one using Valgrind's massif, and targeting functions that allocate too much memory.
--
Kind regards,
Loong Jin
|
affects libindicate
affects dbusmenu
affects indicator-messages
As per the title, these projects leak memory. dbus_g_proxy_end_call requires you
to free the memory it allocates, including the GErrors. The patch attached does
that.
indicator-messages has an additional leak, which I'm not sure is libdbusmenu's
or indicator-messages' fault. libdbusmenu passes a GdkPixbuf into
indicator-messages, indicator-messages scales it and doesn't free the original
pixbuf. However, libdbusmenu doesn't free those pixbufs either, so it gets
leaked. The same thing happened with indicator-sound, but the patch for that has
already been committed.
These patches are by no means complete. I'm just hunting down memleaks one by
one using Valgrind's massif, and targeting functions that allocate too much memory.
--
Kind regards,
Loong Jin
|
|
2010-12-16 00:22:18 |
Chow Loong Jin |
attachment removed |
dbusmenu-memleak-fix.patch https://bugs.launchpad.net/libindicate/+bug/690668/+attachment/1766779/+files/dbusmenu-memleak-fix.patch |
|
|
2010-12-16 00:22:22 |
Chow Loong Jin |
attachment removed |
libindicate-memleak-fix.patch https://bugs.launchpad.net/libindicate/+bug/690668/+attachment/1766781/+files/libindicate-memleak-fix.patch |
|
|
2010-12-16 00:23:47 |
Chow Loong Jin |
attachment added |
|
dbusmenu-memleak-fix.patch https://bugs.launchpad.net/libindicate/+bug/690668/+attachment/1767243/+files/dbusmenu-memleak-fix.patch |
|
2010-12-16 00:24:05 |
Chow Loong Jin |
attachment added |
|
libindicate-memleak-fix.patch https://bugs.launchpad.net/libindicate/+bug/690668/+attachment/1767244/+files/libindicate-memleak-fix.patch |
|
2011-01-21 18:49:04 |
Alex Launi |
indicator-messages: status |
New |
In Progress |
|
2011-01-21 18:49:11 |
Alex Launi |
dbusmenu: status |
New |
Incomplete |
|
2011-01-21 18:49:16 |
Alex Launi |
libindicate: status |
New |
Incomplete |
|
2011-01-21 19:54:31 |
Launchpad Janitor |
branch linked |
|
lp:~ted/libindicate/mem-leak |
|
2011-01-21 20:06:19 |
Launchpad Janitor |
branch linked |
|
lp:~ted/indicator-messages/mem-leaks |
|
2011-01-21 20:09:14 |
Ted Gould |
dbusmenu: status |
Incomplete |
In Progress |
|
2011-01-21 20:09:37 |
Ted Gould |
libindicate: status |
Incomplete |
In Progress |
|
2011-01-21 20:10:27 |
Ted Gould |
branch linked |
|
lp:~ted/dbusmenu/mem-leaks |
|
2011-02-24 18:18:32 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/libindicate |
|
2011-02-24 18:20:49 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/indicator-messages |
|
2011-09-20 03:09:15 |
Ted Gould |
libindicate: status |
In Progress |
Fix Committed |
|
2011-09-20 03:09:17 |
Ted Gould |
libindicate: milestone |
|
0.5.92 |
|
2011-09-20 03:54:11 |
Ted Gould |
libindicate: milestone |
0.5.92 |
0.6.0 |
|
2011-09-20 03:55:32 |
Ted Gould |
bug task added |
|
unity-foundations |
|
2011-09-20 03:55:41 |
Ted Gould |
unity-foundations: status |
New |
In Progress |
|
2011-09-20 03:55:43 |
Ted Gould |
unity-foundations: milestone |
|
oneiric-beta-2 |
|
2011-09-20 04:05:12 |
Ted Gould |
libindicate: status |
Fix Committed |
Fix Released |
|
2011-09-20 04:37:00 |
Launchpad Janitor |
branch linked |
|
lp:~indicator-applet-developers/libindicate/ubuntu |
|
2011-09-20 04:41:10 |
Ted Gould |
dbusmenu: status |
In Progress |
Fix Released |
|
2011-09-20 16:41:31 |
Launchpad Janitor |
branch linked |
|
lp:~ubuntu-desktop/libindicate/ubuntu |
|
2011-09-21 19:47:14 |
Ted Gould |
unity-foundations: milestone |
oneiric-beta-2 |
oneiric-final |
|
2011-09-21 20:15:15 |
Ted Gould |
indicator-messages: status |
In Progress |
Fix Committed |
|
2011-09-21 20:15:19 |
Ted Gould |
indicator-messages: milestone |
|
0.4.96 |
|
2011-09-21 20:15:23 |
Ted Gould |
unity-foundations: status |
In Progress |
Fix Committed |
|
2011-09-23 03:34:23 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/libindicate |
|
2011-09-28 15:32:20 |
Ted Gould |
indicator-messages: milestone |
0.4.96 |
0.5.0 |
|
2011-09-28 15:34:55 |
Ted Gould |
indicator-messages: status |
Fix Committed |
Fix Released |
|
2011-09-28 15:40:53 |
Ted Gould |
unity-foundations: status |
Fix Committed |
Fix Released |
|
2011-09-28 15:59:22 |
Launchpad Janitor |
branch linked |
|
lp:~indicator-applet-developers/indicator-messages/ubuntu |
|
2011-09-28 16:26:24 |
Launchpad Janitor |
branch linked |
|
lp:~ubuntu-desktop/indicator-messages/ubuntu |
|
2011-09-29 01:08:27 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/indicator-messages |
|