The string values have not been checked to be equal to "yes" or
"challenge", but to be other than equal. This is always true because
we compare two different strings. Fix the check, either "yes" or
"challenge" need to be returned to return TRUE now.
---
Something I discovered when I tried to get rid of the Hibernate button
of the logout dialog.

 lxsession-logout/lxsession-logout-dbus-interface.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lxsession-logout/lxsession-logout-dbus-interface.c 
b/lxsession-logout/lxsession-logout-dbus-interface.c
index 3404615..6dde18a 100644
--- a/lxsession-logout/lxsession-logout-dbus-interface.c
+++ b/lxsession-logout/lxsession-logout-dbus-interface.c
@@ -189,7 +189,7 @@ systemd_query (const gchar *function, gboolean 
default_result, GError **error)
     if (g_variant_is_of_type (result, G_VARIANT_TYPE ("(s)")))
     {
                        g_variant_get (result, "(s)", &str);
-                       if (g_strcmp0 (str, "yes") || g_strcmp0 (str, 
"challenge"))
+                       if (!g_strcmp0 (str, "yes") || !g_strcmp0 (str, 
"challenge"))
                                function_result = TRUE;
                        else
                                function_result = default_result;
-- 
2.1.2


------------------------------------------------------------------------------
_______________________________________________
Lxde-list mailing list
Lxde-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxde-list

Reply via email to