aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDima Zavin <dima@android.com>2011-12-30 18:16:44 -0500
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2011-12-30 18:26:34 -0500
commit566cf5b6e34504aaccb76167ecccd7e7e69e6456 (patch)
tree41b3c2b291b96218d92c8a8d321b7400b0a01006
parent36a281e25276f2d138bbbca4170d11453323cce1 (diff)
Input: evdev - only allow reading events if a full packet is present
Without this, it was possible for the reader to get ahead of packet_head. If the input device generated a partial packet *right* after the reader got ahead, then we can get into a situation where the device is marked readable, but read always returns 0 until the next packet is finished (i.e a SYN is generated by the input driver). This situation can also happen if we overflow the buffer while a reader is trying to read an event out. Signed-off-by: Dima Zavin <dima@android.com> Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
-rw-r--r--drivers/input/evdev.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c
index 4cf25347b015..03344b3c7c15 100644
--- a/drivers/input/evdev.c
+++ b/drivers/input/evdev.c
@@ -369,7 +369,7 @@ static int evdev_fetch_next_event(struct evdev_client *client,
369 369
370 spin_lock_irq(&client->buffer_lock); 370 spin_lock_irq(&client->buffer_lock);
371 371
372 have_event = client->head != client->tail; 372 have_event = client->packet_head != client->tail;
373 if (have_event) { 373 if (have_event) {
374 *event = client->buffer[client->tail++]; 374 *event = client->buffer[client->tail++];
375 client->tail &= client->bufsize - 1; 375 client->tail &= client->bufsize - 1;