From d39cf8078e97ac022817c3de2e1fc13c34e19779 Mon Sep 17 00:00:00 2001 From: David Zeuthen Date: Fri, 26 Aug 2011 13:09:40 -0400 Subject: [PATCH] Revert "Support calling device_drive_eject() and device_filesystem_unmount() from daemon" This reverts commit 378a755bbe24e1118caa6a278bcbc585213e7a05. --- src/device.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/device.c b/src/device.c index a928fb5..14d323a 100644 --- a/src/device.c +++ b/src/device.c @@ -6666,8 +6666,7 @@ filesystem_unmount_completed_cb (DBusGMethodInvocation *context, { /* update_info_mount_state() will update the mounts file and clean up the directory if needed */ update_info (device); - if (context != NULL) - dbus_g_method_return (context); + dbus_g_method_return (context); } else { @@ -6806,7 +6805,7 @@ device_filesystem_unmount (Device *device, action_id = NULL; if (!mount_file_has_device (device->priv->device_file, &uid_of_mount, NULL)) { - if (context != NULL && !is_device_in_fstab (device, NULL)) + if (!is_device_in_fstab (device, NULL)) { action_id = "org.freedesktop.udisks.filesystem-unmount-others"; } @@ -6815,7 +6814,7 @@ device_filesystem_unmount (Device *device, { uid_t uid; daemon_local_get_uid (device->priv->daemon, &uid, context); - if (context != NULL && uid_of_mount != uid) + if (uid_of_mount != uid) { action_id = "org.freedesktop.udisks.filesystem-unmount-others"; } @@ -7015,8 +7014,7 @@ drive_eject_completed_cb (DBusGMethodInvocation *context, if (WEXITSTATUS (status) == 0 && !job_was_cancelled) { /* TODO: probably wait for has_media to change to FALSE */ - if (context != NULL) - dbus_g_method_return (context); + dbus_g_method_return (context); } else { @@ -7064,7 +7062,7 @@ device_drive_eject_authorized_cb (Daemon *daemon, if (device_local_is_busy (device, TRUE, &error)) { - throw_error (context, ERROR_BUSY, error->message); + dbus_g_method_return_error (context, error); g_error_free (error); goto out; } @@ -7109,7 +7107,7 @@ device_drive_eject (Device *device, daemon_local_check_auth (device->priv->daemon, device, - context ? "org.freedesktop.udisks.drive-eject" : NULL, + "org.freedesktop.udisks.drive-eject", "DriveEject", TRUE, device_drive_eject_authorized_cb, -- 2.7.4