diff options
author | Robert Ancell <robert.ancell@canonical.com> | 2012-11-30 11:07:01 +0000 |
---|---|---|
committer | Tarmac <Unknown> | 2012-11-30 11:07:01 +0000 |
commit | 821fc4d8c97a016e21422381c820379a958b5091 (patch) | |
tree | ff5c57b39eaac3f560b20554afec4a0a168b5397 /src | |
parent | dabf87de02616a4c0ba54b35dc5cbab8ef0187be (diff) | |
parent | ba4f4df71796d2183171271d4f2cad7998722a71 (diff) | |
download | ayatana-indicator-session-821fc4d8c97a016e21422381c820379a958b5091.tar.gz ayatana-indicator-session-821fc4d8c97a016e21422381c820379a958b5091.tar.bz2 ayatana-indicator-session-821fc4d8c97a016e21422381c820379a958b5091.zip |
Use GDbus. Fixes: https://bugs.launchpad.net/bugs/1084756.
Approved by Charles Kerr.
Diffstat (limited to 'src')
-rw-r--r-- | src/gtk-logout-helper.c | 134 |
1 files changed, 74 insertions, 60 deletions
diff --git a/src/gtk-logout-helper.c b/src/gtk-logout-helper.c index 7868978..55db630 100644 --- a/src/gtk-logout-helper.c +++ b/src/gtk-logout-helper.c @@ -26,28 +26,43 @@ with this program. If not, see <http://www.gnu.org/licenses/>. #include <locale.h> #include <glib.h> #include <glib/gi18n.h> /* textdomain(), bindtextdomain() */ -#include <dbus/dbus-glib.h> #include <gtk/gtk.h> #include "dialog.h" #include "shared-names.h" -static void -consolekit_fallback (LogoutDialogType action) +static GVariant * +call_console_kit (const gchar *method, GVariant *parameters, GError **error) { - g_debug("Falling back to using ConsoleKit for action"); - - DBusGConnection * sbus = dbus_g_bus_get(DBUS_BUS_SYSTEM, NULL); - g_return_if_fail(sbus != NULL); /* worst case */ - DBusGProxy * proxy = dbus_g_proxy_new_for_name(sbus, "org.freedesktop.ConsoleKit", - "/org/freedesktop/ConsoleKit/Manager", - "org.freedesktop.ConsoleKit.Manager"); - - if (proxy == NULL) { - g_warning("Unable to get consolekit proxy"); - return; + GDBusConnection * bus = g_bus_get_sync(G_BUS_TYPE_SYSTEM, NULL, error); + if (!bus) + { + g_variant_unref (parameters); + return NULL; } + GVariant *result = g_dbus_connection_call_sync(bus, + "org.freedesktop.ConsoleKit", + "/org/freedesktop/ConsoleKit/Manager", + "org.freedesktop.ConsoleKit.Manager", + method, + parameters, + NULL, + G_DBUS_CALL_FLAGS_NONE, + -1, + NULL, + error); + g_object_unref (bus); + + return result; +} + +static void +consolekit_fallback (LogoutDialogType action) +{ GError * error = NULL; + GVariant *result = NULL; + + g_debug("Falling back to using ConsoleKit for action"); switch (action) { case LOGOUT_DIALOG_TYPE_LOG_OUT: @@ -55,80 +70,79 @@ consolekit_fallback (LogoutDialogType action) break; case LOGOUT_DIALOG_TYPE_SHUTDOWN: g_debug("Telling ConsoleKit to 'Stop'"); - dbus_g_proxy_call(proxy, - "Stop", - &error, - G_TYPE_INVALID, - G_TYPE_INVALID); + result = call_console_kit ("Stop", g_variant_new ("()"), &error); break; case LOGOUT_DIALOG_TYPE_RESTART: g_debug("Telling ConsoleKit to 'Restart'"); - dbus_g_proxy_call(proxy, - "Restart", - &error, - G_TYPE_INVALID, - G_TYPE_INVALID); + result = call_console_kit ("Restart", g_variant_new ("()"), &error); break; default: g_warning("Unknown action"); break; } - g_object_unref(proxy); + if (!result) { + if (error != NULL) { + g_warning ("ConsoleKit action failed: %s", error->message); + } else { + g_warning ("ConsoleKit action failed: unknown error"); + } - if (error != NULL) { - g_warning("Unable to signal ConsoleKit"); - g_error_free(error); + consolekit_fallback(action); } + else + g_variant_unref (result); + g_clear_error (&error); return; } +static GVariant * +call_gnome_session (const gchar *method, GVariant *parameters, GError **error) +{ + GDBusConnection * bus = g_bus_get_sync(G_BUS_TYPE_SESSION, NULL, error); + if (!bus) + { + g_variant_unref (parameters); + return NULL; + } + + GVariant *result = g_dbus_connection_call_sync(bus, + "org.gnome.SessionManager", + "/org/gnome/SessionManager", + "org.gnome.SessionManager", + method, + parameters, + NULL, + G_DBUS_CALL_FLAGS_NONE, + G_MAXINT, + NULL, + error); + g_object_unref (bus); + + return result; +} + static void session_action (LogoutDialogType action) { - DBusGConnection * sbus; - DBusGProxy * sm_proxy; GError * error = NULL; - gboolean res = FALSE; - - sbus = dbus_g_bus_get(DBUS_BUS_SESSION, NULL); - if (sbus == NULL) { - g_warning("Unable to get DBus session bus."); - return; - } - sm_proxy = dbus_g_proxy_new_for_name_owner (sbus, - "org.gnome.SessionManager", - "/org/gnome/SessionManager", - "org.gnome.SessionManager", - &error); - if (sm_proxy == NULL) { - g_warning("Unable to get DBus proxy to SessionManager interface: %s", error->message); - g_error_free(error); + GVariant *result = NULL; - consolekit_fallback(action); - return; - } - - g_clear_error (&error); - if (action == LOGOUT_DIALOG_TYPE_LOG_OUT) { g_debug("Asking Session manager to 'Logout'"); - res = dbus_g_proxy_call_with_timeout (sm_proxy, "Logout", INT_MAX, &error, - G_TYPE_UINT, 1, G_TYPE_INVALID, G_TYPE_INVALID); + result = call_gnome_session ("Logout", g_variant_new ("(u)", 1), &error); } else if (action == LOGOUT_DIALOG_TYPE_SHUTDOWN) { g_debug("Asking Session manager to 'RequestShutdown'"); - res = dbus_g_proxy_call_with_timeout (sm_proxy, "RequestShutdown", INT_MAX, &error, - G_TYPE_INVALID, G_TYPE_INVALID); + result = call_gnome_session ("RequestShutdown", g_variant_new ("()"), &error); } else if (action == LOGOUT_DIALOG_TYPE_RESTART) { g_debug("Asking Session manager to 'RequestReboot'"); - res = dbus_g_proxy_call_with_timeout (sm_proxy, "RequestReboot", INT_MAX, &error, - G_TYPE_INVALID, G_TYPE_INVALID); + result = call_gnome_session ("RequestReboot", g_variant_new ("()"), &error); } else { g_warning ("Unknown session action"); } - if (!res) { + if (!result) { if (error != NULL) { g_warning ("SessionManager action failed: %s", error->message); } else { @@ -137,8 +151,8 @@ session_action (LogoutDialogType action) consolekit_fallback(action); } - - g_object_unref(sm_proxy); + else + g_variant_unref (result); g_clear_error (&error); return; |