Bug 688640 – Crash in _st_theme_resolve_url

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 688640 - Crash in _st_theme_resolve_url


Summary:	Crash in _st_theme_resolve_url


Status:	RESOLVED DUPLICATE of bug 689029

Product:	gnome-shell
Classification:	Core
Component:	general
Version:	3.6.x
Hardware:	Other Linux

Importance:	Urgent critical
Target Milestone:	---
Assigned To:	gnome-shell-maint
QA Contact:	gnome-shell-maint

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2012-11-19 14:04 UTC by Guillaume Desmottes
Modified:	2012-11-28 12:17 UTC

See Also:
GNOME target:	---
GNOME version:	---

Description Guillaume Desmottes 2012-11-19 14:04:36 UTC

This morning I updated to Shell master (5eae1851a1db9a7e52bdc0d6d6eec7baf36008a4) and since experience this crash at least one or 2 time each hour.

Program received signal SIGSEGV, Segmentation fault.
0x00007f2c7b9003f9 in _st_theme_resolve_url (theme=0x0, base_stylesheet=0x3b17550, url=0x3b88e60 "running-indicator.svg") at st/st-theme.c:1057
1057	  base_filename = g_hash_table_lookup (theme->filenames_by_stylesheet, base_stylesheet);

+ Trace 231192

#0 _st_theme_resolve_url
at st/st-theme.c line 1057
#1 _st_theme_node_ensure_background
at st/st-theme-node.c line 1853
#2 st_theme_node_paint_equal
at st/st-theme-node.c line 3609
#3 st_widget_recompute_style
at st/st-widget.c line 1547
#4 st_widget_style_changed
at st/st-widget.c line 540
#5 notify_children_of_style_change
at st/st-widget.c line 507
#6 st_widget_real_style_changed
at st/st-widget.c line 523
#7 st_button_style_changed
at st/st-button.c line 113
#8 g_cclosure_marshal_VOID__VOIDv
at gmarshal.c line 115
#9 g_type_class_meta_marshalv
at gclosure.c line 997
#10 _g_closure_invoke_va
at gclosure.c line 840
#11 g_signal_emit_valist
at gsignal.c line 3226
#12 g_signal_emit
at gsignal.c line 3371
#13 st_widget_recompute_style
at st/st-widget.c line 1585
#14 st_widget_ensure_style
at st/st-widget.c line 1602
#15 st_widget_map
at st/st-widget.c line 482
#16 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#17 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#18 clutter_actor_realize
at ./clutter-actor.c line 1926
#19 clutter_actor_update_map_state
at ./clutter-actor.c line 1431
#20 clutter_actor_map
at ./clutter-actor.c line 1525
#21 clutter_actor_real_map
at ./clutter-actor.c line 1493
#22 st_widget_map
at st/st-widget.c line 480
#23 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#24 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#25 clutter_actor_realize
at ./clutter-actor.c line 1926
#26 clutter_actor_update_map_state
at ./clutter-actor.c line 1431
#27 clutter_actor_map
at ./clutter-actor.c line 1525
#28 clutter_actor_real_map
at ./clutter-actor.c line 1493
#29 st_widget_map
at st/st-widget.c line 480
#30 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#31 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#32 clutter_actor_map
at ./clutter-actor.c line 1525
#33 clutter_actor_real_map
at ./clutter-actor.c line 1493
#34 st_widget_map
at st/st-widget.c line 480
#35 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#36 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#37 clutter_actor_map
at ./clutter-actor.c line 1525
#38 clutter_actor_real_map
at ./clutter-actor.c line 1493
#39 st_widget_map
at st/st-widget.c line 480
#40 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#41 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#42 clutter_actor_map
at ./clutter-actor.c line 1525
#43 clutter_actor_real_map
at ./clutter-actor.c line 1493
#44 st_widget_map
at st/st-widget.c line 480
#45 clutter_actor_set_mapped
at ./clutter-actor.c line 1244
#46 clutter_actor_update_map_state
at ./clutter-actor.c line 1454
#47 clutter_actor_real_show
at ./clutter-actor.c line 1625
#48 g_cclosure_marshal_VOID__VOIDv
at gmarshal.c line 115
#49 g_type_class_meta_marshalv
at gclosure.c line 997
#50 _g_closure_invoke_va
at gclosure.c line 840
#51 g_signal_emit_valist
at gsignal.c line 3226
#52 g_signal_emit
at gsignal.c line 3371
#53 clutter_actor_show
at ./clutter-actor.c line 1716
#54 ffi_call_unix64
at ../src/x86/unix64.S line 75
#55 ffi_call
at ../src/x86/ffi64.c line 486
#56 gjs_invoke_c_function
at gi/function.c line 891
#57 function_call
at gi/function.c line 1202
#58 CallJSNative
at jscntxtinlines.h line 701
#59 js::Invoke
at jsinterp.cpp line 696
#60 js::Interpret
at jsinterp.cpp line 4810
#61 js::RunScript
at jsinterp.cpp line 653
#62 js::Invoke
at jsinterp.cpp line 740
#63 js_fun_apply
at jsfun.cpp line 2205
#64 CallJSNative
at jscntxtinlines.h line 701
#65 js::Interpret
at jsinterp.cpp line 4799
#66 js::RunScript
at jsinterp.cpp line 653
#67 js::Invoke
at jsinterp.cpp line 740
#68 js_fun_apply
at jsfun.cpp line 2205
#69 CallJSNative
at jscntxtinlines.h line 701
#70 js::Interpret
at jsinterp.cpp line 4799
#71 js::RunScript
at jsinterp.cpp line 653
#72 js::Invoke
at jsinterp.cpp line 740
#73 js_fun_apply
at jsfun.cpp line 2205
#74 CallJSNative
at jscntxtinlines.h line 701
#75 js::Interpret
at jsinterp.cpp line 4799
#76 js::RunScript
at jsinterp.cpp line 653
#77 js::Invoke
at jsinterp.cpp line 740
#78 js_fun_apply
at jsfun.cpp line 2205
#79 CallJSNative
at jscntxtinlines.h line 701
#80 js::Interpret
at jsinterp.cpp line 4799
#81 js::RunScript
at jsinterp.cpp line 653
#82 js::Invoke
at jsinterp.cpp line 740
#83 js::CallOrConstructBoundFunction
at jsfun.cpp line 2319
#84 CallJSNative
at jscntxtinlines.h line 701
#85 js::Invoke
at jsinterp.cpp line 703
#86 js::ExternalInvoke
at jsinterp.cpp line 863
#87 JS_CallFunctionValue
at jsapi.cpp line 5145
#88 gjs_call_function_value
at gjs/jsapi-util.c line 1180
#89 gjs_closure_invoke
at gi/closure.c line 278
#90 closure_marshal
at gi/value.c line 127
#91 g_closure_invoke
at gclosure.c line 777
#92 signal_emit_unlocked_R
at gsignal.c line 3567
#93 g_signal_emit_valist
at gsignal.c line 3325
#94 g_signal_emit
at gsignal.c line 3371
#95 clutter_actor_event
at ./clutter-actor.c line 13520
#96 _clutter_actor_handle_event
at ./clutter-actor.c line 19952
#97 emit_event_chain
at ./clutter-main.c line 2290
#98 emit_pointer_event
at ./clutter-main.c line 2310
#99 _clutter_process_event_details
at ./clutter-main.c line 2487
#100 _clutter_process_event
at ./clutter-main.c line 2777
#101 _clutter_stage_process_queued_events
at ./clutter-stage.c line 1026
#102 master_clock_process_events
at ./clutter-master-clock.c line 283
#103 clutter_clock_dispatch
at ./clutter-master-clock.c line 514
#104 g_main_dispatch
at gmain.c line 2715
#105 g_main_context_dispatch
at gmain.c line 3219
#106 g_main_context_iterate
at gmain.c line 3290
#107 g_main_loop_run
at gmain.c line 3484
#108 meta_run
at core/main.c line 545
#109 main
at main.c line 414


	Inferior 1 [process 2429] will be detached.

Quit anyway? (y or n) Detaching from program: /home/cassidy/usr/bin/gnome-shell, process 2429

Comment 1 Rico Tzschichholz 2012-11-19 19:03:28 UTC

It seems to be introduced by the 3 committed patches proposed in https://bugzilla.gnome.org/show_bug.cgi?id=687465

Comment 2 Rico Tzschichholz 2012-11-21 08:29:55 UTC

I got this crash despite reverting those commits. Although it seems to be harder to trigger it.

Comment 3 Guillaume Desmottes 2012-11-26 15:36:07 UTC

Raising the importance as the Shell is barely usable atm. I get this crash at least 20 times per day.

Comment 4 Jasper St. Pierre (not reading bugmail) 2012-11-26 16:45:35 UTC

Can you try the patch in bug #689029 ? It shouldn't do anything, but according to Giovanni, it does something.

Comment 5 Stéphane Démurget 2012-11-27 12:27:33 UTC

I had the crash many times each day.

For me this is a duplicate of 689029 as the latest patch in bug 689029 fixes the issue, as st_theme_node_dispose was called and was clearing node->theme

Comment 6 Guillaume Desmottes 2012-11-28 12:17:32 UTC

Yeah, it seems fixed now. Thanks!

*** This bug has been marked as a duplicate of bug 689029 ***