! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
xfce4-session hanging on log out
Status:
RESOLVED: FIXED
Severity:
critical
Product:
Xfce4-session
Component:
General

Comments

Description Andre Miranda editbugs 2017-08-19 21:34:31 CEST
After updating all core components to current master, session is hanging indefinitely on shutdown, restart, log out.
I can't find any suspect error message on log, only:
(xfce4-session:4300): Gtk-CRITICAL **: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed
(xfce4-session:4300): GLib-GObject-WARNING **: invalid (NULL) pointer instance
(xfce4-session:4300): GLib-GObject-CRITICAL **: g_signal_emit_valist: assertion 'G_TYPE_CHECK_INSTANCE (instance)' failed

Sometimes I can't even switch to another TTY to kill xfce4-session, but I'm not sure this is related or another component is the culprit.
Comment 1 Andre Miranda editbugs 2017-08-20 17:17:17 CEST
With the help of ninetls, we're able to find out that Thunar is the culprit, killing it makes session finish almost instantaneously. However, when I reverted to session 4.12, Thunar hangs for about 8 seconds and then the session is finished, so it leads me to think that we hit a regression, now session waits indefinitely for the processes to quit.
Comment 2 Viktor Odintsev editbugs 2017-08-20 17:51:25 CEST
Created attachment 7290 
xfsm-manager: pass manager variable to g_timeout_add

g_timeout_add now takes manager_quit_signal instead gtk_main_quit. In this function XfsmManager instance is used to emit MANAGER_QUIT signal which was forgotten to be passed to g_timeout_add.

This patch fixes this problem, now Xfsm should shutdown the session after 7 seconds as it was before.
Comment 3 Git Bot editbugs 2017-08-21 17:42:58 CEST
Viktor Odintsev referenced this bugreport in commit b9cfbafe7cc9517dc424212aca0199f037eb977d

xfsm-manager: pass manager variable to g_timeout_add (Bug #13802)

https://git.xfce.org/xfce/xfce4-session/commit?id=b9cfbafe7cc9517dc424212aca0199f037eb977d
Comment 4 Eric Koegel editbugs 2017-08-22 12:05:53 CEST
Thanks Viktor, I pushed it to master and gave you credit for your work!

Bug #13802

Reported by:
Andre Miranda
Reported on: 2017-08-19
Last modified on: 2017-08-22

People

Assignee:
Xfce Bug Triage
CC List:
4 users

Version

Version:
4.13.0

Attachments

xfsm-manager: pass manager variable to g_timeout_add (892 bytes, patch)
2017-08-20 17:51 CEST , Viktor Odintsev
ninetls : review? ( eric )

Additional information