Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1402426 > unrolled thread
| Started by | Ricky Liang <jcliang@chromium.org> |
|---|---|
| First post | 2016-05-17 17:50 +0200 |
| Last post | 2016-05-20 20:00 +0200 |
| Articles | 5 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] Input: uinput - handle compat ioctl for UI_SET_PHYS Ricky Liang <jcliang@chromium.org> - 2016-05-17 17:50 +0200
Re: [PATCH] Input: uinput - handle compat ioctl for UI_SET_PHYS Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-05-20 18:40 +0200
Re: [PATCH] Input: uinput - handle compat ioctl for UI_SET_PHYS Ricky Liang <jcliang@chromium.org> - 2016-05-20 20:00 +0200
Re: [PATCH v2] Input: uinput - handle compat ioctl for UI_SET_PHYS Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-05-20 20:00 +0200
[PATCH v2] Input: uinput - handle compat ioctl for UI_SET_PHYS Ricky Liang <jcliang@chromium.org> - 2016-05-20 20:00 +0200
| From | Ricky Liang <jcliang@chromium.org> |
|---|---|
| Date | 2016-05-17 17:50 +0200 |
| Subject | [PATCH] Input: uinput - handle compat ioctl for UI_SET_PHYS |
| Message-ID | <rzPpU-6oq-13@gated-at.bofh.it> |
When running a 32-bit userspace on a 64-bit kernel, the UI_SET_PHYS
ioctl needs to be treated with special care, as it has the pointer
size encoded in the command.
Signed-off-by: Ricky Liang <jcliang@chromium.org>
---
drivers/input/misc/uinput.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c
index abe1a92..b4d1b1d 100644
--- a/drivers/input/misc/uinput.c
+++ b/drivers/input/misc/uinput.c
@@ -984,6 +984,15 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
static long uinput_compat_ioctl(struct file *file,
unsigned int cmd, unsigned long arg)
{
+ switch (_IOC_NR(cmd)) {
+ case _IOC_NR(UI_SET_PHYS):
+ if (_IOC_SIZE(cmd) == sizeof(compat_uptr_t)) {
+ cmd &= ~IOCSIZE_MASK;
+ cmd |= sizeof(void *) << IOCSIZE_SHIFT;
+ }
+ break;
+ }
+
return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg));
}
#endif
--
2.1.2
[toc] | [next] | [standalone]
| From | Dmitry Torokhov <dmitry.torokhov@gmail.com> |
|---|---|
| Date | 2016-05-20 18:40 +0200 |
| Message-ID | <rAVCW-7On-15@gated-at.bofh.it> |
| In reply to | #1402426 |
Hi Ricky,
On Tue, May 17, 2016 at 11:39:45PM +0800, Ricky Liang wrote:
> When running a 32-bit userspace on a 64-bit kernel, the UI_SET_PHYS
> ioctl needs to be treated with special care, as it has the pointer
> size encoded in the command.
>
> Signed-off-by: Ricky Liang <jcliang@chromium.org>
> ---
> drivers/input/misc/uinput.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c
> index abe1a92..b4d1b1d 100644
> --- a/drivers/input/misc/uinput.c
> +++ b/drivers/input/misc/uinput.c
> @@ -984,6 +984,15 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
> static long uinput_compat_ioctl(struct file *file,
> unsigned int cmd, unsigned long arg)
> {
> + switch (_IOC_NR(cmd)) {
> + case _IOC_NR(UI_SET_PHYS):
> + if (_IOC_SIZE(cmd) == sizeof(compat_uptr_t)) {
> + cmd &= ~IOCSIZE_MASK;
> + cmd |= sizeof(void *) << IOCSIZE_SHIFT;
> + }
> + break;
> + }
> +
This looks quite complicated... Can we do this:
#define UI_SET_PHYS_COMPAT __IOW(UINPUT_IOCTL_BASE, 108, compat_uptr_t)
...
if (cmd == UI_SET_PHYS_COMPAT)
cmd = UI_SET_PHYS;
> return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg));
> }
> #endif
We can use the local define instead of manipulating cmd size because we
will never going to change UI_SET_PHYS definition, since it is part of
uapi.
Thanks.
--
Dmitry
[toc] | [prev] | [next] | [standalone]
| From | Ricky Liang <jcliang@chromium.org> |
|---|---|
| Date | 2016-05-20 20:00 +0200 |
| Message-ID | <rAWSl-dY-15@gated-at.bofh.it> |
| In reply to | #1404591 |
Hi Dmitry,
On Sat, May 21, 2016 at 12:32 AM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> Hi Ricky,
>
> On Tue, May 17, 2016 at 11:39:45PM +0800, Ricky Liang wrote:
>> When running a 32-bit userspace on a 64-bit kernel, the UI_SET_PHYS
>> ioctl needs to be treated with special care, as it has the pointer
>> size encoded in the command.
>>
>> Signed-off-by: Ricky Liang <jcliang@chromium.org>
>> ---
>> drivers/input/misc/uinput.c | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c
>> index abe1a92..b4d1b1d 100644
>> --- a/drivers/input/misc/uinput.c
>> +++ b/drivers/input/misc/uinput.c
>> @@ -984,6 +984,15 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
>> static long uinput_compat_ioctl(struct file *file,
>> unsigned int cmd, unsigned long arg)
>> {
>> + switch (_IOC_NR(cmd)) {
>> + case _IOC_NR(UI_SET_PHYS):
>> + if (_IOC_SIZE(cmd) == sizeof(compat_uptr_t)) {
>> + cmd &= ~IOCSIZE_MASK;
>> + cmd |= sizeof(void *) << IOCSIZE_SHIFT;
>> + }
>> + break;
>> + }
>> +
>
> This looks quite complicated... Can we do this:
>
> #define UI_SET_PHYS_COMPAT __IOW(UINPUT_IOCTL_BASE, 108, compat_uptr_t)
>
> ...
>
> if (cmd == UI_SET_PHYS_COMPAT)
> cmd = UI_SET_PHYS;
>
>> return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg));
>> }
>> #endif
>
> We can use the local define instead of manipulating cmd size because we
> will never going to change UI_SET_PHYS definition, since it is part of
> uapi.
Sounds good. I'll send v2 to implement this.
Thanks,
Ricky
>
> Thanks.
>
> --
> Dmitry
[toc] | [prev] | [next] | [standalone]
| From | Dmitry Torokhov <dmitry.torokhov@gmail.com> |
|---|---|
| Date | 2016-05-20 20:00 +0200 |
| Subject | Re: [PATCH v2] Input: uinput - handle compat ioctl for UI_SET_PHYS |
| Message-ID | <rAWSm-dY-25@gated-at.bofh.it> |
| In reply to | #1402426 |
On Sat, May 21, 2016 at 01:50:46AM +0800, Ricky Liang wrote:
> When running a 32-bit userspace on a 64-bit kernel, the UI_SET_PHYS
> ioctl needs to be treated with special care, as it has the pointer
> size encoded in the command.
>
> Signed-off-by: Ricky Liang <jcliang@chromium.org>
Applied, thank you.
> ---
> drivers/input/misc/uinput.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c
> index abe1a92..65ebbd1 100644
> --- a/drivers/input/misc/uinput.c
> +++ b/drivers/input/misc/uinput.c
> @@ -981,9 +981,15 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
> }
>
> #ifdef CONFIG_COMPAT
> +
> +#define UI_SET_PHYS_COMPAT _IOW(UINPUT_IOCTL_BASE, 108, compat_uptr_t)
> +
> static long uinput_compat_ioctl(struct file *file,
> unsigned int cmd, unsigned long arg)
> {
> + if (cmd == UI_SET_PHYS_COMPAT)
> + cmd = UI_SET_PHYS;
> +
> return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg));
> }
> #endif
> --
> 2.1.2
>
--
Dmitry
[toc] | [prev] | [next] | [standalone]
| From | Ricky Liang <jcliang@chromium.org> |
|---|---|
| Date | 2016-05-20 20:00 +0200 |
| Subject | [PATCH v2] Input: uinput - handle compat ioctl for UI_SET_PHYS |
| Message-ID | <rAWSm-dY-27@gated-at.bofh.it> |
| In reply to | #1402426 |
When running a 32-bit userspace on a 64-bit kernel, the UI_SET_PHYS
ioctl needs to be treated with special care, as it has the pointer
size encoded in the command.
Signed-off-by: Ricky Liang <jcliang@chromium.org>
---
drivers/input/misc/uinput.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c
index abe1a92..65ebbd1 100644
--- a/drivers/input/misc/uinput.c
+++ b/drivers/input/misc/uinput.c
@@ -981,9 +981,15 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
}
#ifdef CONFIG_COMPAT
+
+#define UI_SET_PHYS_COMPAT _IOW(UINPUT_IOCTL_BASE, 108, compat_uptr_t)
+
static long uinput_compat_ioctl(struct file *file,
unsigned int cmd, unsigned long arg)
{
+ if (cmd == UI_SET_PHYS_COMPAT)
+ cmd = UI_SET_PHYS;
+
return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg));
}
#endif
--
2.1.2
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web