Thanks for the patch. Is it the same bug / patch that
http://sourceforge.net/p/lxde/bugs/616/ ?
Regards,
Julien Lavergne
2014-10-29 15:38 GMT+01:00 Stefan Agner <ste...@agner.ch>:
> 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
