The Sound Menu

memleak with album art resizing

Bug #689686 reported by Conor Curran
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Sound Menu
Fix Released
Medium
Conor Curran
The Sound Menu 0.5.3
indicator-sound (Ubuntu)
Fix Released
Undecided
Unassigned
Maverick
Fix Released
Undecided
Unassigned

Bug Description

metadata-widget.c

metatdata_image_expose method - line 200

      pixbuf = gdk_pixbuf_scale_simple(pixbuf,60, 60, GDK_INTERP_BILINEAR);

The original data in pixbuf (the original album art) gets lost for ever. After running for a long time the mem leak can become a problem.

Conor Curran (cjcurran)
Changed in indicator-sound:
milestone: none → 0.5.3
importance: Undecided → Medium
status: New → Fix Committed
assignee: nobody → Conor Curran (cjcurran)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

For regression testing, restart your session (logout/login) and play a song in rhythmbox. You should still see the album art in the sound indicator.

Changed in indicator-sound (Ubuntu):
status: New → Fix Released
Changed in indicator-sound:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted indicator-sound into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in indicator-sound (Ubuntu Maverick):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
jhfhlkjlj (fdsuufijjejejejej-deactivatedaccount) wrote :

It seems to work fine.

Omer Akram (om26er)
tags: added: verification-done
removed: verification-needed
Conor Curran (cjcurran)
Changed in indicator-sound (Ubuntu Maverick):
status: Fix Committed → Fix Released
Revision history for this message
Omer Akram (om26er) wrote :

proposed version have not yet reached -updates ;)

Changed in indicator-sound (Ubuntu Maverick):
status: Fix Released → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-sound - 0.5.0.1-0ubuntu2

---------------
indicator-sound (0.5.0.1-0ubuntu2) maverick-proposed; urgency=low

  * debian/patches/metadata-widget.patch
    - Fixed memory leak with album art resizing (LP: #689686)
 -- Ken VanDine <email address hidden> Thu, 16 Dec 2010 20:24:05 -0500

Changed in indicator-sound (Ubuntu Maverick):
status: Fix Committed → 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.