After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 732952 - crash in update_card (gvc-mixer-control.c) when using pulseaudio-5.0
crash in update_card (gvc-mixer-control.c) when using pulseaudio-5.0
Status: RESOLVED NOTGNOME
Product: libgnome-volume-control
Classification: Other
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: libgnome-volume-control-maint
libgnome-volume-control-maint
Depends on:
Blocks:
 
 
Reported: 2014-07-09 15:38 UTC by Rex Dieter
Modified: 2014-07-10 12:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
quick-n-dirty workaround for crash condition (922 bytes, patch)
2014-07-09 15:38 UTC, Rex Dieter
none Details | Review

Description Rex Dieter 2014-07-09 15:38:40 UTC
Created attachment 280284 [details] [review]
quick-n-dirty workaround for crash condition

Seeing this downstream in fedora,
https://bugzilla.redhat.com/show_bug.cgi?id=1117683

that gvc-mixer-control.c tries to query profile names (for g_debug calls mostly?), and these names sometimes happen to be NULL when using pulseaudio-5.0, so crashes happen.

Attached is a quick-n-dirty workaround to avoid this situation, but probably worthwhile digging deeper to find why the profile names aren't getting set.
Comment 1 Giovanni Campagna 2014-07-09 15:49:26 UTC
Shouldn't pulseaudio be fixed instead, to provide names that are not NULL?
Comment 2 Rex Dieter 2014-07-09 16:02:12 UTC
Quite possible, I've filed a bug against pulseaudio (pavucontrol) too,
https://bugs.freedesktop.org/show_bug.cgi?id=81116
Comment 3 Rex Dieter 2014-07-10 12:00:40 UTC
OK, pulseaudio dev onlist agrees with that approach (of fixing pulseaudio), let's close this (for now).