Ubuntu
notification-daemon package

Notification daemon leaks X11 windows

Bug #102128 reported by Mathias Hasselmann
6
Affects Status Importance Assigned to Milestone
notification-daemon (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: notification-daemon

With every notification show, notification-daemon 0.3.6-0ubuntu2 leaks two X11 windows as the following commands demonstrate:

    notify-send "$(date)"
    xwininfo -root -children | grep notification-daemon | wc -l

The problem seems to be fixed in upstream version of notification-daemon 0.3.7.
Christian Hammond suggests to upgrade libnotify and notification-daemon anyway,
as releases 0.4.4 and 0.3.7 contain "lots of bug fixed and memory leak fixes".

Maybe solving this ticked also solves #67129.

Revision history for this message
Mathias Hasselmann (hasselmm) wrote :

Duh, I was not carefully enough: After applying 01_ubuntu_theme.patch and 02_make_ubuntu_theme_default.patch to notification-deamon 0.3.7 it also is leaking.

Revision history for this message
Mathias Hasselmann (hasselmm) wrote :

This patch fixed the leak for me. Maybe it makes sense to run valgrind on notification-deamon with ubunu theme, as notification-daemon is used frequently in ubuntu.

didier (did447-deactivatedaccount)
Changed in notification-daemon:
status: Unconfirmed → Confirmed
Revision history for this message
Mathias Hasselmann (hasselmm) wrote :

Chris moved some folders, so the patches have to be patched for 0.3.7.

Revision history for this message
Mathias Hasselmann (hasselmm) wrote :
Revision history for this message
Mathias Hasselmann (hasselmm) wrote :
Revision history for this message
Mathias Hasselmann (hasselmm) wrote :

This patch also required some code changes to apply.

Revision history for this message
Mathias Hasselmann (hasselmm) wrote :
Revision history for this message
Mathias Hasselmann (hasselmm) wrote :

Once again 102128.diff. 06_fix_prefix_for_dbus_binding_tool.patch is not needed anymore.

Revision history for this message
Mörgæs (moergaes) wrote :

Closing because of 'The problem seems to be fixed in upstream version of notification-daemon 0.3.7' in original post.

Changed in notification-daemon (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.