Bug 594906 – empathy crashed with SIGSEGV in empathy_audio_sink_set_volume()

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 594906 - empathy crashed with SIGSEGV in empathy_audio_sink_set_volume()


Summary:	empathy crashed with SIGSEGV in empathy_audio_sink_set_volume()


Status:	RESOLVED FIXED

Product:	empathy
Classification:	Core
Component:	VoIP
Version:	2.27.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	empathy-maint
QA Contact:

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2009-09-11 16:25 UTC by Sebastien Bacher
Modified:	2009-09-14 14:46 UTC

See Also:
GNOME target:	2.28.x
GNOME version:	---

Description Sebastien Bacher 2009-09-11 16:25:18 UTC

The bug has been opened on https://bugs.launchpad.net/bugs/427579

"Crash during the connection process of an audio call.
Crash occures 100% time when an audio call is open.

empathy 2.27.92-0ubuntu2

+ Trace 217498

#0 empathy_audio_sink_set_volume
at empathy-audio-sink.c line 223
#1 IA__g_cclosure_marshal_VOID__DOUBLE
at /build/buildd/glib2.0-2.21.6/gobject/gmarshal.c line 461
#2 IA__g_closure_invoke
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 767
#3 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3247
#4 IA__g_signal_emit_valist
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 2980
#5 IA__g_signal_emit
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3037
#6 gtk_scale_button_scale_value_changed
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkscalebutton.c line 1574
#7 IA__g_cclosure_marshal_VOID__VOID
at /build/buildd/glib2.0-2.21.6/gobject/gmarshal.c line 77
#8 g_type_class_meta_marshal
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 878
#9 IA__g_closure_invoke
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 767
#10 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3285
#11 IA__g_signal_emit_valist
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 2980
#12 IA__g_signal_emit
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3037
#13 gtk_range_adjustment_value_changed
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 2487
#14 IA__g_cclosure_marshal_VOID__VOID
at /build/buildd/glib2.0-2.21.6/gobject/gmarshal.c line 77
#15 IA__g_closure_invoke
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 767
#16 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3247
#17 IA__g_signal_emit_valist
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 2980
#18 IA__g_signal_emit
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3037
#19 IA__gtk_adjustment_value_changed
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkadjustment.c line 668
#20 gtk_range_real_change_value
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 3514
#21 _gtk_marshal_BOOLEAN__ENUM_DOUBLE
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkmarshalers.c line 245
#22 g_type_class_meta_marshal
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 878
#23 IA__g_closure_invoke
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 767
#24 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3285
#25 IA__g_signal_emit_valist
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 2990
#26 IA__g_signal_emit
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3037
#27 page_back
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 2554
#28 gtk_range_scroll
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 2663
#29 gtk_range_add_step_timer
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 3601
#30 gtk_range_button_press
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkrange.c line 2065
#31 gtk_scale_button_scale_press
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkscalebutton.c line 1450
#32 _gtk_marshal_BOOLEAN__BOXED
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkmarshalers.c line 84
#33 g_type_class_meta_marshal
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 878
#34 IA__g_closure_invoke
at /build/buildd/glib2.0-2.21.6/gobject/gclosure.c line 767
#35 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3285
#36 IA__g_signal_emit_valist
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 2990
#37 IA__g_signal_emit
at /build/buildd/glib2.0-2.21.6/gobject/gsignal.c line 3037
#38 gtk_widget_event_internal
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkwidget.c line 4767
#39 IA__gtk_propagate_event
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkmain.c line 2396
#40 IA__gtk_main_do_event
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkmain.c line 1601
#41 gdk_event_dispatch
at /build/buildd/gtk+2.0-2.17.11/gdk/x11/gdkevents-x11.c line 2382
#42 IA__g_main_context_dispatch
at /build/buildd/glib2.0-2.21.6/glib/gmain.c line 1960
#43 g_main_context_iterate
at /build/buildd/glib2.0-2.21.6/glib/gmain.c line 2591
#44 IA__g_main_loop_run
at /build/buildd/glib2.0-2.21.6/glib/gmain.c line 2799
#45 IA__gtk_main
at /build/buildd/gtk+2.0-2.17.11/gtk/gtkmain.c line 1205
#46 main
at empathy.c line 909

Comment 1 Guillaume Desmottes 2009-09-14 13:53:14 UTC

empathy_audio_sink_set_volume is called on a NULL EmpathyGstAudioSink. Not sure how that can happen. Would be good to have Empathy logs. Could you please start empathy with EMPATHY_DEBUG=all ?

Comment 2 Cosimo Cecchi 2009-09-14 14:29:53 UTC

I can't reproduce the bug, but ut seems that when initializing the call window, the "value-changed" signal on the volume slider is connected before the audio sink is initialized.
My branch here [1] fixes this, and adds a NULL check in the callback. Not sure if this was really the cause for the crash though.

[1] http://git.collabora.co.uk/?p=user/cosimoc/empathy.git;a=shortlog;h=refs/heads/toolbar_setup

Comment 3 Cosimo Cecchi 2009-09-14 14:46:42 UTC

The branch is now merged to master.