From 415c6d2e2742946606c4db5b1157a946767afdb7 Mon Sep 17 00:00:00 2001 From: Giovanni Campagna Date: Fri, 25 Oct 2013 18:02:16 +0200 Subject: [PATCH] gapplication: don't emit warnings for DBus calls If a broken (or malicious) remote client calls Open or CommandLine on a GApplication that does not implement those, return a DBus error instead of going through and then emitting a warning. https://bugzilla.gnome.org/show_bug.cgi?id=710885 --- gio/gapplicationimpl-dbus.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/gio/gapplicationimpl-dbus.c b/gio/gapplicationimpl-dbus.c index 07709cc9e..2c77fee81 100644 --- a/gio/gapplicationimpl-dbus.c +++ b/gio/gapplicationimpl-dbus.c @@ -203,12 +203,20 @@ g_application_impl_method_call (GDBusConnection *connection, else if (strcmp (method_name, "Open") == 0) { + GApplicationFlags flags; GVariant *platform_data; const gchar *hint; GVariant *array; GFile **files; gint n, i; + flags = g_application_get_flags (impl->app); + if ((flags & G_APPLICATION_HANDLES_OPEN) == 0) + { + g_dbus_method_invocation_return_error (invocation, G_DBUS_ERROR, G_DBUS_ERROR_NOT_SUPPORTED, "Application does not open files"); + return; + } + /* freedesktop interface has no hint parameter */ if (g_str_equal (interface_name, "org.freedesktop.Application")) { @@ -246,10 +254,18 @@ g_application_impl_method_call (GDBusConnection *connection, else if (strcmp (method_name, "CommandLine") == 0) { + GApplicationFlags flags; GApplicationCommandLine *cmdline; GVariant *platform_data; int status; + flags = g_application_get_flags (impl->app); + if ((flags & G_APPLICATION_HANDLES_COMMAND_LINE) == 0) + { + g_dbus_method_invocation_return_error (invocation, G_DBUS_ERROR, G_DBUS_ERROR_NOT_SUPPORTED, "Application does not handle command line arguments"); + return; + } + /* Only on the GtkApplication interface */ cmdline = g_dbus_command_line_new (invocation);