Hi Julien, Yes, thats exactly it!
-- Stefan Am 2014-10-31 16:16, schrieb Julien Lavergne: > 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
