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
This commit is contained in:
Giovanni Campagna 2013-10-25 18:02:16 +02:00
parent 3e48e3be40
commit 415c6d2e27

View File

@ -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);