Commit a6706174 authored by Takashi Iwai's avatar Takashi Iwai Committed by Ben Hutchings

Input: evdev - do not report errors form flush()

commit eb38f3a4 upstream.

We've got bug reports showing the old systemd-logind (at least
system-210) aborting unexpectedly, and this turned out to be because
of an invalid error code from close() call to evdev devices.  close()
is supposed to return only either EINTR or EBADFD, while the device
returned ENODEV.  logind was overreacting to it and decided to kill
itself when an unexpected error code was received.  What a tragedy.

The bad error code comes from flush fops, and actually evdev_flush()
returns ENODEV when device is disconnected or client's access to it is
revoked. But in these cases the fact that flush did not actually happen is
not an error, but rather normal behavior. For non-disconnected devices
result of flush is also not that interesting as there is no potential of
data loss and even if it fails application has no way of handling the
error. Because of that we are better off always returning success from
evdev_flush().

Also returning EINTR from flush()/close() is discouraged (as it is not
clear how application should handle this error), so let's stop taking
evdev->mutex interruptibly.

Bugzilla: http://bugzilla.suse.com/show_bug.cgi?id=939834Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
[bwh: Backported to 3.2: there's no revoked flag to test]
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent 7808f78e
...@@ -126,19 +126,14 @@ static int evdev_flush(struct file *file, fl_owner_t id) ...@@ -126,19 +126,14 @@ static int evdev_flush(struct file *file, fl_owner_t id)
{ {
struct evdev_client *client = file->private_data; struct evdev_client *client = file->private_data;
struct evdev *evdev = client->evdev; struct evdev *evdev = client->evdev;
int retval;
retval = mutex_lock_interruptible(&evdev->mutex); mutex_lock(&evdev->mutex);
if (retval)
return retval;
if (!evdev->exist) if (evdev->exist)
retval = -ENODEV; input_flush_device(&evdev->handle, file);
else
retval = input_flush_device(&evdev->handle, file);
mutex_unlock(&evdev->mutex); mutex_unlock(&evdev->mutex);
return retval; return 0;
} }
static void evdev_free(struct device *dev) static void evdev_free(struct device *dev)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment