Not a fix, but a workaround: Each time app_indicator_set_menu() is
called, we leak by 200k. so, I reduced the number of times it gets
called. (Currently called any time a device is updated, which seems to
happen every 30 seconds. Devices are updated even if nothing has
changed.)

I'm rather novice, so I've probably fudged something else up by doing
this. It works for me (desktop, wireless mouse being the only battery
indicator I have) I couldn't make heads or tails of the actual leak
itself, though.

The attached patch has been applied to package, and uploaded to my 
ppa:aaron-haviland/ppa for Maverick.
I've also applied it to the Lucid package and uploaded, but I don't have a 
lucid system to test it on.

After running for 2 hours, according to /proc/pid/smaps Rss has only
increased by 240kB

** Patch added: "15-bug569273-leak.patch"
   
https://bugs.launchpad.net/ubuntu/maverick/+source/gnome-power-manager/+bug/569273/+attachment/1760173/+files/15-bug569273-leak.patch

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/569273

Title:
  memory leak in gnome-power-manager

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to