Comment 85 for bug 569273

Revision history for this message
Aaron Haviland (aaron-haviland) wrote :

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