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 786603 - Crashes when alt-tabbing away from window
Crashes when alt-tabbing away from window
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
git master
Other Linux
: Normal critical
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
: 786547 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2017-08-22 12:44 UTC by Bastien Nocera
Modified: 2017-08-23 12:34 UTC
See Also:
GNOME target: 3.26
GNOME version: ---


Attachments
keyboard: Fix warning when typing something (3.49 KB, patch)
2017-08-22 15:47 UTC, Bastien Nocera
rejected Details | Review
printers: Disconnect the search bar from the shell's key-press-event (1.27 KB, patch)
2017-08-22 18:00 UTC, Rui Matos
committed Details | Review

Description Bastien Nocera 2017-08-22 12:44:06 UTC
I switched through all the panels. This happened when I tried to Alt-Tab away from the shell.

My guess is that one of the panels didn't destroy its search bar properly.

  • #0 gtk_search_bar_handle_event
    at gtksearchbar.c line 218
  • #1 _gtk_marshal_BOOLEAN__BOXED
    at gtkmarshalers.c line 82
  • #2 g_closure_invoke
    at /home/hadess/Projects/jhbuild/glib/gobject/gclosure.c line 804
  • #3 signal_emit_unlocked_R
    at /home/hadess/Projects/jhbuild/glib/gobject/gsignal.c line 3635
  • #4 g_signal_emit_valist
    at /home/hadess/Projects/jhbuild/glib/gobject/gsignal.c line 3401
  • #5 g_signal_emit
    at /home/hadess/Projects/jhbuild/glib/gobject/gsignal.c line 3447
  • #6 gtk_widget_event_internal
    at gtkwidget.c line 7723
  • #7 propagate_event
    at gtkmain.c line 2671
  • #8 gtk_main_do_event
    at gtkmain.c line 1911
  • #9 _gdk_event_emit
    at /home/hadess/Projects/jhbuild/gtk+-3/gdk/gdkevents.c line 73
  • #10 gdk_event_source_dispatch
    at /home/hadess/Projects/jhbuild/gtk+-3/gdk/wayland/gdkeventsource.c line 121
  • #11 g_main_dispatch
    at /home/hadess/Projects/jhbuild/glib/glib/gmain.c line 3237
  • #12 g_main_context_dispatch
    at /home/hadess/Projects/jhbuild/glib/glib/gmain.c line 3912
  • #13 g_main_context_iterate
    at /home/hadess/Projects/jhbuild/glib/glib/gmain.c line 3985
  • #14 g_main_context_iteration
    at /home/hadess/Projects/jhbuild/glib/glib/gmain.c line 4046
  • #15 g_application_run
    at /home/hadess/Projects/jhbuild/glib/gio/gapplication.c line 2378
  • #16 main
    at main.c line 57

Comment 1 Bastien Nocera 2017-08-22 13:04:11 UTC
Reproducer:
Open the Keyboard panel, go to another panel, and type stuff...
Comment 2 Bastien Nocera 2017-08-22 15:47:39 UTC
Created attachment 358166 [details] [review]
keyboard: Fix warning when typing something

(gnome-control-center:5955): Gtk-WARNING **: The search bar does not have an entry connected to it. Call gtk_search_bar_connect_entry() to connect one.

This also fixes this crash after the keyboard panel has been closed:
 #0  0x00007f5d861aaacf in gtk_search_bar_handle_event (bar=bar@entry=0x2225f30, event=0x2be55a0) at gtksearchbar.c:218
 #1  0x00007f5d862e2eeb in _gtk_marshal_BOOLEAN__BOXED (closure=0x2ae28e0, return_value=0x7ffc06868840, n_param_values=<optimized out>, param_values=0x7ffc06868930, invocation_hint=<optimized out>,
     marshal_data=<optimized out>) at gtkmarshalers.c:82
 #2  0x00007f5d84983ac6 in g_closure_invoke (closure=0x2ae28e0, return_value=0x7ffc06868840, n_param_values=2, param_values=0x7ffc06868930, invocation_hint=0x7ffc06868870)
     at ../jhbuild/glib/gobject/gclosure.c:804
 #3  0x00007f5d849a17cf in signal_emit_unlocked_R (node=0x1909a10, detail=0, instance=0x1f18310, emission_return=0x7ffc06868a10, instance_and_params=0x7ffc06868930)
     at ../jhbuild/glib/gobject/gsignal.c:3635
 #4  0x00007f5d849a0b5b in g_signal_emit_valist (instance=0x1f18310, signal_id=90, detail=0, var_args=0x7ffc06868bf8) at ../jhbuild/glib/gobject/gsignal.c:3401
 #5  0x00007f5d849a1048 in g_signal_emit (instance=0x1f18310, signal_id=90, detail=0) at ../jhbuild/glib/gobject/gsignal.c:3447
 #6  0x00007f5d8642d6e4 in gtk_widget_event_internal (widget=0x1f18310, event=0x2be55a0) at gtkwidget.c:7723
 #7  0x00007f5d862dff8f in propagate_event (widget=0x1f18310, event=0x2be55a0, captured=<optimized out>, topmost=0x0) at gtkmain.c:2671
 #8  0x00007f5d862e1fe0 in gtk_main_do_event (event=<optimized out>) at gtkmain.c:1911
 #9  0x00007f5d85dca6c2 in _gdk_event_emit (event=0x2be55a0) at ../jhbuild/gtk+-3/gdk/gdkevents.c:73
 #10 0x00007f5d85e49ae9 in gdk_event_source_dispatch (base=0x18ad410, callback=0x0, data=0x0) at ../jhbuild/gtk+-3/gdk/wayland/gdkeventsource.c:121
 #11 0x00007f5d8469a882 in g_main_dispatch (context=0x18a3ee0) at ../jhbuild/glib/glib/gmain.c:3237
 #12 0x00007f5d8469b7f7 in g_main_context_dispatch (context=0x18a3ee0) at ../jhbuild/glib/glib/gmain.c:3912
 #13 0x00007f5d8469b9dc in g_main_context_iterate (context=0x18a3ee0, block=1, dispatch=1, self=0x1891750) at ../jhbuild/glib/glib/gmain.c:3985
 #14 0x00007f5d8469baa0 in g_main_context_iteration (context=0x18a3ee0, may_block=1) at ../jhbuild/glib/glib/gmain.c:4046
 #15 0x00007f5d84c93088 in g_application_run (application=0x19108e0, argc=1, argv=0x7ffc06869138) at ../jhbuild/glib/gio/gapplication.c:2378
 #16 0x000000000044c4ff in main (argc=1, argv=0x7ffc06869138) at main.c:57
Comment 3 Rui Matos 2017-08-22 17:28:08 UTC
*** Bug 786547 has been marked as a duplicate of this bug. ***
Comment 4 Rui Matos 2017-08-22 17:46:25 UTC
Review of attachment 358166 [details] [review]:

hmm, I don't get that warning and I still get the crash with this patch
Comment 5 Rui Matos 2017-08-22 18:00:08 UTC
Created attachment 358175 [details] [review]
printers: Disconnect the search bar from the shell's key-press-event

Using g_connect_object() ensures the disconnection happens when the
search bar is finalized.
--

This seems to fix it for me
Comment 6 Bastien Nocera 2017-08-22 18:37:15 UTC
Review of attachment 358175 [details] [review]:

Better than my version that does nothing!
Comment 7 Bastien Nocera 2017-08-22 18:37:33 UTC
Review of attachment 358166 [details] [review]:

This does nothing.
Comment 8 Rui Matos 2017-08-23 12:34:25 UTC
Attachment 358175 [details] pushed as 535c5d1 - printers: Disconnect the search bar from the shell's key-press-event