Commit ab4e0192 authored by Dmitry Torokhov's avatar Dmitry Torokhov
Browse files

Input: define separate EVIOCGKEYCODE_V2/EVIOCSKEYCODE_V2 



The desire to keep old names for the EVIOCGKEYCODE/EVIOCSKEYCODE while
extending them to support large scancodes was a mistake. While we tried
to keep ABI intact (and we succeeded in doing that, programs compiled
on older kernels will work on newer ones) there is still a problem with
recompiling existing software with newer kernel headers.

New kernel headers will supply updated ioctl numbers and kernel will
expect that userspace will use struct input_keymap_entry to set and
retrieve keymap data. But since the names of ioctls are still the same
userspace will happily compile even if not adjusted to make use of the
new structure and will start miraculously fail in the field.

To avoid this issue let's revert EVIOCGKEYCODE/EVIOCSKEYCODE definitions
and add EVIOCGKEYCODE_V2/EVIOCSKEYCODE_V2 so that userspace can explicitly
select the style of ioctls it wants to employ.

Reviewed-by: default avatarHenrik Rydberg <rydberg@euromail.se>
Acked-by: default avatarJarod Wilson <jarod@redhat.com>
Acked-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
Signed-off-by: default avatarDmitry Torokhov <dtor@mail.ru>
parent 57a7872f

drivers/input/evdev.c

+58 −55
Original line number Original line Diff line number Diff line
@@ -534,24 +534,19 @@ static int handle_eviocgbit(struct input_dev *dev, 
}

}

#undef OLD_KEY_MAX

#undef OLD_KEY_MAX





static int evdev_handle_get_keycode(struct input_dev *dev,

static int evdev_handle_get_keycode(struct input_dev *dev, void __user *p)

				    void __user *p, size_t size)

{

{

	struct input_keymap_entry ke;

	struct input_keymap_entry ke = {

		.len	= sizeof(unsigned int),

		.flags	= 0,

	};

	int __user *ip = (int __user *)p;

	int error;

	int error;





	memset(&ke, 0, sizeof(ke));



	if (size == sizeof(unsigned int[2])) {

	/* legacy case */

	/* legacy case */

		int __user *ip = (int __user *)p;



	if (copy_from_user(ke.scancode, p, sizeof(unsigned int)))

	if (copy_from_user(ke.scancode, p, sizeof(unsigned int)))

		return -EFAULT;

		return -EFAULT;





		ke.len = sizeof(unsigned int);

		ke.flags = 0;



	error = input_get_keycode(dev, &ke);

	error = input_get_keycode(dev, &ke);

	if (error)

	if (error)

		return error;

		return error;
@@ -559,31 +554,33 @@ static int evdev_handle_get_keycode(struct input_dev *dev, 
	if (put_user(ke.keycode, ip + 1))

	if (put_user(ke.keycode, ip + 1))

		return -EFAULT;

		return -EFAULT;





	} else {

	return 0;

		size = min(size, sizeof(ke));

}



static int evdev_handle_get_keycode_v2(struct input_dev *dev, void __user *p)

{

	struct input_keymap_entry ke;

	int error;





		if (copy_from_user(&ke, p, size))

	if (copy_from_user(&ke, p, sizeof(ke)))

		return -EFAULT;

		return -EFAULT;





	error = input_get_keycode(dev, &ke);

	error = input_get_keycode(dev, &ke);

	if (error)

	if (error)

		return error;

		return error;





		if (copy_to_user(p, &ke, size))

	if (copy_to_user(p, &ke, sizeof(ke)))

		return -EFAULT;

		return -EFAULT;

	}



	return 0;

	return 0;

}

}





static int evdev_handle_set_keycode(struct input_dev *dev,

static int evdev_handle_set_keycode(struct input_dev *dev, void __user *p)

				    void __user *p, size_t size)

{

{

	struct input_keymap_entry ke;

	struct input_keymap_entry ke = {



		.len	= sizeof(unsigned int),

	memset(&ke, 0, sizeof(ke));

		.flags	= 0,



	};

	if (size == sizeof(unsigned int[2])) {

		/* legacy case */

	int __user *ip = (int __user *)p;

	int __user *ip = (int __user *)p;





	if (copy_from_user(ke.scancode, p, sizeof(unsigned int)))

	if (copy_from_user(ke.scancode, p, sizeof(unsigned int)))
@@ -592,18 +589,18 @@ static int evdev_handle_set_keycode(struct input_dev *dev, 
	if (get_user(ke.keycode, ip + 1))

	if (get_user(ke.keycode, ip + 1))

		return -EFAULT;

		return -EFAULT;





		ke.len = sizeof(unsigned int);

	return input_set_keycode(dev, &ke);

		ke.flags = 0;

}





	} else {

static int evdev_handle_set_keycode_v2(struct input_dev *dev, void __user *p)

		size = min(size, sizeof(ke));

{

	struct input_keymap_entry ke;





		if (copy_from_user(&ke, p, size))

	if (copy_from_user(&ke, p, sizeof(ke)))

		return -EFAULT;

		return -EFAULT;





	if (ke.len > sizeof(ke.scancode))

	if (ke.len > sizeof(ke.scancode))

		return -EINVAL;

		return -EINVAL;

	}





	return input_set_keycode(dev, &ke);

	return input_set_keycode(dev, &ke);

}

}
@@ -669,6 +666,18 @@ static long evdev_do_ioctl(struct file *file, unsigned int cmd, 
			return evdev_grab(evdev, client);

			return evdev_grab(evdev, client);

		else

		else

			return evdev_ungrab(evdev, client);

			return evdev_ungrab(evdev, client);



	case EVIOCGKEYCODE:

		return evdev_handle_get_keycode(dev, p);



	case EVIOCSKEYCODE:

		return evdev_handle_set_keycode(dev, p);



	case EVIOCGKEYCODE_V2:

		return evdev_handle_get_keycode_v2(dev, p);



	case EVIOCSKEYCODE_V2:

		return evdev_handle_set_keycode_v2(dev, p);

	}

	}





	size = _IOC_SIZE(cmd);

	size = _IOC_SIZE(cmd);
@@ -708,12 +717,6 @@ static long evdev_do_ioctl(struct file *file, unsigned int cmd, 
			return -EFAULT;

			return -EFAULT;





		return error;

		return error;



	case EVIOC_MASK_SIZE(EVIOCGKEYCODE):

		return evdev_handle_get_keycode(dev, p, size);



	case EVIOC_MASK_SIZE(EVIOCSKEYCODE):

		return evdev_handle_set_keycode(dev, p, size);

	}

	}





	/* Multi-number variable-length handlers */

	/* Multi-number variable-length handlers */

include/linux/input.h

+4 −2
Original line number Original line Diff line number Diff line
@@ -104,8 +104,10 @@ struct input_keymap_entry { 
#define EVIOCGREP		_IOR('E', 0x03, unsigned int[2])	/* get repeat settings */

#define EVIOCGREP		_IOR('E', 0x03, unsigned int[2])	/* get repeat settings */

#define EVIOCSREP		_IOW('E', 0x03, unsigned int[2])	/* set repeat settings */

#define EVIOCSREP		_IOW('E', 0x03, unsigned int[2])	/* set repeat settings */





#define EVIOCGKEYCODE		_IOR('E', 0x04, struct input_keymap_entry)	/* get keycode */

#define EVIOCGKEYCODE		_IOR('E', 0x04, unsigned int[2])        /* get keycode */

#define EVIOCSKEYCODE		_IOW('E', 0x04, struct input_keymap_entry)	/* set keycode */

#define EVIOCGKEYCODE_V2	_IOR('E', 0x04, struct input_keymap_entry)

#define EVIOCSKEYCODE		_IOW('E', 0x04, unsigned int[2])        /* set keycode */

#define EVIOCSKEYCODE_V2	_IOW('E', 0x04, struct input_keymap_entry)





#define EVIOCGNAME(len)		_IOC(_IOC_READ, 'E', 0x06, len)		/* get device name */

#define EVIOCGNAME(len)		_IOC(_IOC_READ, 'E', 0x06, len)		/* get device name */

#define EVIOCGPHYS(len)		_IOC(_IOC_READ, 'E', 0x07, len)		/* get physical location */

#define EVIOCGPHYS(len)		_IOC(_IOC_READ, 'E', 0x07, len)		/* get physical location */

CRA Git | Maintained and supported by SUSTech CRA and CCSE