[Spice-devel] [PATCH] console-kit: Check session-lock just by IdleHint

Pavel Grunt pgrunt at redhat.com
Tue Jan 17 13:45:58 UTC 2017


gnome-shell in RHEL6 does not emit the Unlock signal when session
is being unlocked making the agent think that the session is locked.

Remove handling of Lock/Unlock signals and keep only IdleHintChanged.

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1412673
---
signal handling can be removed as well
---
 src/vdagentd/console-kit.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/vdagentd/console-kit.c b/src/vdagentd/console-kit.c
index 024a260..5230508 100644
--- a/src/vdagentd/console-kit.c
+++ b/src/vdagentd/console-kit.c
@@ -493,11 +493,10 @@ gboolean session_info_session_is_locked(struct session_info *info)
 
     /* Not every system does emit Lock and Unlock signals (for instance, such
      * is the case for RHEL6) but most of the systems seems to emit the
-     * IdleHintChanged. So, we give priority to the Lock signal, if it is Locked
-     * we return that the session is locked, otherwise we double check with the
-     * IdleHint value */
+     * IdleHintChanged. So use the IdleHint value.
+     * systemd-login uses locked-hint which is not implemented in ConsoleKit. */
     si_dbus_read_signals(info);
-    locked = (info->session_is_locked || info->session_idle_hint);
+    locked = info->session_idle_hint;
     if (info->verbose) {
         syslog(LOG_DEBUG, "(console-kit) session is locked: %s",
                locked ? "yes" : "no");
-- 
2.11.0



More information about the Spice-devel mailing list