Comment 2 for bug 1391649

Revision history for this message
Jason Conti (jconti) wrote :

Turns out there was a pretty serious memory leak in notification_new_from_dbus_message. I didn't think I owned the references to the GVariants, but as it turns out, I did.

Before the fix valgrind was telling me we were leaking about 1M per 1000 notifications.

I have pushed the fix out for 14.04, 15.04 and 15.10. I can push it to older versions if necessary, not sure which ones still have support.

Again apologizes about missing this bug for so long.