Bug 621109 – gnome-appearance-properties crashed with SIGSEGV in cursor_size_scale_value_changed_cb()

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 621109 - gnome-appearance-properties crashed with SIGSEGV in cursor_size_scale_value_changed_cb()


Summary:	gnome-appearance-properties crashed with SIGSEGV in cursor_size_scale_value_c...


Status:	RESOLVED INCOMPLETE

Product:	gnome-control-center
Classification:	Core
Component:	[obsolete] Appearance
Version:	2.30.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	Control-Center Maintainers
QA Contact:	Control-Center Maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2010-06-09 13:24 UTC by Pedro Villavicencio
Modified:	2010-11-09 22:52 UTC

See Also:
GNOME target:	---
GNOME version:	2.29/2.30

Description Pedro Villavicencio 2010-06-09 13:24:48 UTC

this report has been filed here:

https://bugs.edge.launchpad.net/ubuntu/+source/gnome-control-center/+bug/450443

"crash when i change mouse pointer type and size once."

".

+ Trace 222325

Thread 1 (process 2286)

#0 cursor_size_scale_value_changed_cb
at appearance-style.c line 475
#1 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#2 ??
from /usr/lib/libgobject-2.0.so.0
#3 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#4 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#5 gtk_range_adjustment_value_changed
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkrange.c line 2637
#6 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#7 ??
from /usr/lib/libgobject-2.0.so.0
#8 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#9 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#10 IA__gtk_adjustment_value_changed
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkadjustment.c line 668
#11 gtk_range_real_change_value
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkrange.c line 3664
#12 _gtk_marshal_BOOLEAN__ENUM_DOUBLE
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkmarshalers.c line 245
#13 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#14 ??
from /usr/lib/libgobject-2.0.so.0
#15 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#16 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#17 update_slider_position
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkrange.c line 2352
#18 gtk_range_motion_notify
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkrange.c line 2495
#19 _gtk_marshal_BOOLEAN__BOXED
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkmarshalers.c line 84
#20 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#21 ??
from /usr/lib/libgobject-2.0.so.0
#22 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#23 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#24 gtk_widget_event_internal
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkwidget.c line 4951
#25 IA__gtk_propagate_event
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkmain.c line 2447
#26 IA__gtk_main_do_event
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkmain.c line 1647
#27 gdk_event_dispatch
at /build/buildd/gtk+2.0-2.20.0/gdk/x11/gdkevents-x11.c line 2372
#28 g_main_context_dispatch
from /lib/libglib-2.0.so.0
#29 ??
from /lib/libglib-2.0.so.0
#30 g_main_loop_run
from /lib/libglib-2.0.so.0
#31 IA__gtk_main
at /build/buildd/gtk+2.0-2.20.0/gtk/gtkmain.c line 1219
#32 main
at appearance-main.c line 249

Comment 1 Jens Granseuer 2010-06-12 09:04:16 UTC

> cursor_size_scale_value_changed_cb
> at appearance-style.c line 475

That line number doesn't seem to make sense when looking at 2.28. What's the code, and are you applying custom patches?

Comment 2 Pedro Villavicencio 2010-06-14 12:26:38 UTC

appearance-style.c is being modified for the compiz integration, but the cursor* part is not being touched there, the patch is available here: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/lucid/gnome-control-center/lucid/annotate/head:/debian/patches/95_desktop-effects-integration.patch ; Thanks for looking into this Jens!.

Comment 3 Sebastien Bacher 2010-06-14 12:36:35 UTC

l475 is the gconf_client_set_int() call in

"#ifdef HAVE_XCURSOR
static void
cursor_size_changed_cb (int size, AppearanceData *data)
{
  gconf_client_set_int (data->client, CURSOR_SIZE_KEY, size, NULL);"

Comment 4 Sebastien Bacher 2010-06-14 12:38:25 UTC

the line is the same in git we don't change this source

Comment 5 Jens Granseuer 2010-06-14 14:40:17 UTC

That makes even less sense, Sebastien. First of all, that line is 476, and secondly the trace says cursor_size_scale_value_changed_cb which is, um, not cursor_size_changed_cb.

The patch pointed out by Pedro doesn't touch that code. The stack trace, however, seems to be somewhat unreliable.

Comment 6 Sebastien Bacher 2010-06-14 19:59:00 UTC

> the trace says cursor_size_scale_value_changed_cb which is, um, not
cursor_size_changed_cb.

welcome to stacktrace from optimized (-02) code builds... I will try to see if I can get a valgrind log or a stacktrace using O0 later

Comment 7 Tobias Mueller 2010-11-09 22:52:38 UTC

Closing this bug report as no further information has been provided. Please feel free to reopen this bug if you can provide the information asked for.
Thanks!