summaryrefslogtreecommitdiffstats
path: root/drivers/input
diff options
context:
space:
mode:
authorIan Campbell <ijc@hellion.org.uk>2005-09-04 01:41:14 -0500
committerDmitry Torokhov <dtor_core@ameritech.net>2005-09-04 01:41:14 -0500
commit4cee99564db7f65a6f88e4b752da52768cde3802 (patch)
tree6c355494c3c399d2c1fc2d746171f434ded907e4 /drivers/input
parentd2b5ffca73594e4046f405e3ef2438ce41f76fb2 (diff)
downloadop-kernel-dev-4cee99564db7f65a6f88e4b752da52768cde3802.zip
op-kernel-dev-4cee99564db7f65a6f88e4b752da52768cde3802.tar.gz
Input: fix checking whether new keycode fits size-wise
When dev->keycodesize == sizeof(int) the old code produces incorrect result. Signed-off-by: Ian Campbell <ijc@hellion.org.uk> Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/input')
-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 20e3a16..3a8314b 100644
--- a/drivers/input/evdev.c
+++ b/drivers/input/evdev.c
@@ -320,7 +320,7 @@ static long evdev_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
if (t < 0 || t >= dev->keycodemax || !dev->keycodesize) return -EINVAL;
if (get_user(v, ip + 1)) return -EFAULT;
if (v < 0 || v > KEY_MAX) return -EINVAL;
- if (v >> (dev->keycodesize * 8)) return -EINVAL;
+ if (dev->keycodesize < sizeof(v) && (v >> (dev->keycodesize * 8))) return -EINVAL;
u = SET_INPUT_KEYCODE(dev, t, v);
clear_bit(u, dev->keybit);
set_bit(v, dev->keybit);
OpenPOWER on IntegriCloud