indicator-sound-service crashed with SIGSEGV in g_icon_serialize()

Bug #1229612 reported by Ricardo J. Moreira Teixeira on 2013-09-24
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Sound Menu
Fix Released
Medium
Charles Kerr
indicator-sound (Ubuntu)
Medium
Unassigned

Bug Description

When I open beatbox player the sound icon disappears and it gives me this error

ProblemType: Crash
DistroRelease: Ubuntu 13.10
Package: indicator-sound 12.10.2+13.10.20130918-0ubuntu1
ProcVersionSignature: Ubuntu 3.11.0-7.14-generic 3.11.1
Uname: Linux 3.11.0-7-generic i686
ApportVersion: 2.12.4-0ubuntu1
Architecture: i386
Date: Tue Sep 24 11:57:05 2013
ExecutablePath: /usr/lib/i386-linux-gnu/indicator-sound/indicator-sound-service
InstallationDate: Installed on 2012-08-04 (416 days ago)
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha i386 (20120730.1)
MarkForUpload: True
ProcCmdline: /usr/lib/i386-linux-gnu/indicator-sound/indicator-sound-service
SegvAnalysis:
 Segfault happened at: 0xb760d4df <g_icon_serialize+31>: mov 0x0(%ebp),%eax
 PC (0xb760d4df) ok
 source "0x0(%ebp)" (0x00000000) not located in a known VMA region (needed readable region)!
 destination "%eax" ok
SegvReason: reading NULL VMA
Signal: 11
SourcePackage: indicator-sound
StacktraceTop:
 g_icon_serialize () from /usr/lib/i386-linux-gnu/libgio-2.0.so.0
 sound_menu_add_player ()
 g_hash_table_foreach () from /lib/i386-linux-gnu/libglib-2.0.so.0
 ?? ()
 g_cclosure_marshal_VOID__OBJECT () from /usr/lib/i386-linux-gnu/libgobject-2.0.so.0
Title: indicator-sound-service crashed with SIGSEGV in g_icon_serialize()
UpgradeStatus: Upgraded to saucy on 2013-08-26 (28 days ago)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo

Related branches

StacktraceTop:
 g_icon_serialize (icon=0x0) at /build/buildd/glib2.0-2.37.93/./gio/gicon.c:665
 sound_menu_add_player (self=0x9bf3b50, player=0x9c132f0) at /build/buildd/indicator-sound-12.10.2+13.10.20130918/obj-i686-linux-gnu/src/sound-menu.c:340
 g_hash_table_foreach (hash_table=0x9be9448, func=func@entry=0x804fc40 <___lambda2__gh_func>, user_data=user_data@entry=0x9c13c00) at /build/buildd/glib2.0-2.37.93/./glib/ghash.c:1526
 indicator_sound_service_player_added (player=0x9c132f0, self=0x9bef020) at /build/buildd/indicator-sound-12.10.2+13.10.20130918/obj-i686-linux-gnu/src/service.c:1773
 _indicator_sound_service_player_added_media_player_list_player_added (_sender=0x9beeb98, player=0x9c132f0, self=0x9bef020) at /build/buildd/indicator-sound-12.10.2+13.10.20130918/obj-i686-linux-gnu/src/service.c:350

Changed in indicator-sound (Ubuntu):
importance: Undecided → Medium
tags: removed: need-i386-retrace
information type: Private → Public
Charles Kerr (charlesk) wrote :

Looks like DesktopAppInfo.get_icon() is returning NULL when called by MediaPlayer's "icon" property getter, and we're passing that NULL into g_icon_serialize().

Changed in indicator-sound:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Charles Kerr (charlesk)
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-sound at revision 398, scheduled for release in indicator-sound, milestone Unknown

Changed in indicator-sound:
status: In Progress → Fix Committed
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-sound/13.10 at revision None, scheduled for release in indicator-sound, milestone 0.5.6

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-sound - 12.10.2+14.04.20131029.1-0ubuntu1

---------------
indicator-sound (12.10.2+14.04.20131029.1-0ubuntu1) trusty; urgency=low

  [ Dmitrijs Ledkovs ]
  * Define "ubiquity" indicator profile, reusing the greeter object. (LP:
    #1241539)

  [ Charles Kerr ]
  * When we can't connect to pulse, wait a moment before retrying. (LP:
    #1244010)
  * In Service's volume_changed() function, explicitly instantiate the
    GVariant to avoid leaking it. See the description in bug #1244468
    for before & after C code and more information. (LP: #1244468)
  * Test that MediaPlayer's icon property isn't NULL before we serialize
    it. (LP: #1229612)

  [ Sebastien Bacher ]
  * Don't special case Unity sessions, that's not needed since gnome-
    control-center is doing the right thing, the session name used was
    also wrong and couldn't work (lp: #1239545). (LP: #1239545)

  [ Lars Uebernickel ]
  * Only display running players in the phone's sound menu The design
    says to only display the _one_ running player. This will have that
    effect, as there's only one player allowed to run at a time on the
    phone.
  * VolumeControl: don't pass vala closures into libpulse Libpulse's
    vala bindings don't specify that callbacks passed into some of its
    functions (e.g., get_server_info) can be called after the function
    returns. Vala thus frees closure data after these functions return.
    This can't easily be fixed in the bindings, because libpulse doesn't
    provide variants of these functions with destroy_notifies. This
    patch works around this problem by only passing non-closure
    functions into libpulse. (LP: #1244386)
  * Depend on vala 0.20 and use its GIcon.serialize().

  [ Ted Gould ]
  * Adding a phone_greeter profile.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 399
 -- Ubuntu daily release <email address hidden> Tue, 29 Oct 2013 10:32:48 +0000

Changed in indicator-sound (Ubuntu):
status: New → Fix Released
Charles Kerr (charlesk) on 2014-03-13
Changed in indicator-sound:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers