* [PATCH v2 01/24] Input: input_event: fix struct padding on sparc64 [not found] <20191213204936.3643476-1-arnd@arndb.de> @ 2019-12-13 20:49 ` Arnd Bergmann 2019-12-13 22:08 ` Dmitry Torokhov 0 siblings, 1 reply; 2+ messages in thread From: Arnd Bergmann @ 2019-12-13 20:49 UTC (permalink / raw) To: y2038, linux-kernel, Dmitry Torokhov, Deepa Dinamani Cc: Arnd Bergmann, sparclinux, David S. Miller, stable, Bjorn Helgaas, Lubomir Rintel, Thomas Gleixner, linux-input Going through all uses of timeval, I noticed that we screwed up input_event in the previous attempts to fix it: The time fields now match between kernel and user space, but all following fields are in the wrong place. Add the required padding that is implied by the glibc timeval definition to fix the layout, and use a struct initializer to avoid leaking kernel stack data. Cc: sparclinux@vger.kernel.org Cc: "David S. Miller" <davem@davemloft.net> Fixes: 141e5dcaa735 ("Input: input_event - fix the CONFIG_SPARC64 mixup") Fixes: 2e746942ebac ("Input: input_event - provide override for sparc64") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/input/evdev.c | 14 +++++++------- drivers/input/misc/uinput.c | 14 +++++++++----- include/uapi/linux/input.h | 1 + 3 files changed, 17 insertions(+), 12 deletions(-) diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c index d7dd6fcf2db0..f918fca9ada3 100644 --- a/drivers/input/evdev.c +++ b/drivers/input/evdev.c @@ -224,13 +224,13 @@ static void __pass_event(struct evdev_client *client, */ client->tail = (client->head - 2) & (client->bufsize - 1); - client->buffer[client->tail].input_event_sec = - event->input_event_sec; - client->buffer[client->tail].input_event_usec = - event->input_event_usec; - client->buffer[client->tail].type = EV_SYN; - client->buffer[client->tail].code = SYN_DROPPED; - client->buffer[client->tail].value = 0; + client->buffer[client->tail] = (struct input_event) { + .input_event_sec = event->input_event_sec, + .input_event_usec = event->input_event_usec, + .type = EV_SYN, + .code = SYN_DROPPED, + .value = 0, + }; client->packet_head = client->tail; } diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c index fd253781be71..2dabbe47d43e 100644 --- a/drivers/input/misc/uinput.c +++ b/drivers/input/misc/uinput.c @@ -74,12 +74,16 @@ static int uinput_dev_event(struct input_dev *dev, struct uinput_device *udev = input_get_drvdata(dev); struct timespec64 ts; - udev->buff[udev->head].type = type; - udev->buff[udev->head].code = code; - udev->buff[udev->head].value = value; ktime_get_ts64(&ts); - udev->buff[udev->head].input_event_sec = ts.tv_sec; - udev->buff[udev->head].input_event_usec = ts.tv_nsec / NSEC_PER_USEC; + + udev->buff[udev->head] = (struct input_event) { + .input_event_sec = ts.tv_sec, + .input_event_usec = ts.tv_nsec / NSEC_PER_USEC, + .type = type, + .code = code, + .value = value, + }; + udev->head = (udev->head + 1) % UINPUT_BUFFER_SIZE; wake_up_interruptible(&udev->waitq); diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index f056b2a00d5c..9a61c28ed3ae 100644 --- a/include/uapi/linux/input.h +++ b/include/uapi/linux/input.h @@ -34,6 +34,7 @@ struct input_event { __kernel_ulong_t __sec; #if defined(__sparc__) && defined(__arch64__) unsigned int __usec; + unsigned int __pad; #else __kernel_ulong_t __usec; #endif -- 2.20.0 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2 01/24] Input: input_event: fix struct padding on sparc64 2019-12-13 20:49 ` [PATCH v2 01/24] Input: input_event: fix struct padding on sparc64 Arnd Bergmann @ 2019-12-13 22:08 ` Dmitry Torokhov 0 siblings, 0 replies; 2+ messages in thread From: Dmitry Torokhov @ 2019-12-13 22:08 UTC (permalink / raw) To: Arnd Bergmann Cc: y2038, linux-kernel, Deepa Dinamani, sparclinux, David S. Miller, stable, Bjorn Helgaas, Lubomir Rintel, Thomas Gleixner, linux-input On Fri, Dec 13, 2019 at 09:49:10PM +0100, Arnd Bergmann wrote: > Going through all uses of timeval, I noticed that we screwed up > input_event in the previous attempts to fix it: > > The time fields now match between kernel and user space, but > all following fields are in the wrong place. > > Add the required padding that is implied by the glibc timeval > definition to fix the layout, and use a struct initializer > to avoid leaking kernel stack data. > > Cc: sparclinux@vger.kernel.org > Cc: "David S. Miller" <davem@davemloft.net> > Fixes: 141e5dcaa735 ("Input: input_event - fix the CONFIG_SPARC64 mixup") > Fixes: 2e746942ebac ("Input: input_event - provide override for sparc64") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Applied, thank you. > --- > drivers/input/evdev.c | 14 +++++++------- > drivers/input/misc/uinput.c | 14 +++++++++----- > include/uapi/linux/input.h | 1 + > 3 files changed, 17 insertions(+), 12 deletions(-) > > diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c > index d7dd6fcf2db0..f918fca9ada3 100644 > --- a/drivers/input/evdev.c > +++ b/drivers/input/evdev.c > @@ -224,13 +224,13 @@ static void __pass_event(struct evdev_client *client, > */ > client->tail = (client->head - 2) & (client->bufsize - 1); > > - client->buffer[client->tail].input_event_sec = > - event->input_event_sec; > - client->buffer[client->tail].input_event_usec = > - event->input_event_usec; > - client->buffer[client->tail].type = EV_SYN; > - client->buffer[client->tail].code = SYN_DROPPED; > - client->buffer[client->tail].value = 0; > + client->buffer[client->tail] = (struct input_event) { > + .input_event_sec = event->input_event_sec, > + .input_event_usec = event->input_event_usec, > + .type = EV_SYN, > + .code = SYN_DROPPED, > + .value = 0, > + }; > > client->packet_head = client->tail; > } > diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c > index fd253781be71..2dabbe47d43e 100644 > --- a/drivers/input/misc/uinput.c > +++ b/drivers/input/misc/uinput.c > @@ -74,12 +74,16 @@ static int uinput_dev_event(struct input_dev *dev, > struct uinput_device *udev = input_get_drvdata(dev); > struct timespec64 ts; > > - udev->buff[udev->head].type = type; > - udev->buff[udev->head].code = code; > - udev->buff[udev->head].value = value; > ktime_get_ts64(&ts); > - udev->buff[udev->head].input_event_sec = ts.tv_sec; > - udev->buff[udev->head].input_event_usec = ts.tv_nsec / NSEC_PER_USEC; > + > + udev->buff[udev->head] = (struct input_event) { > + .input_event_sec = ts.tv_sec, > + .input_event_usec = ts.tv_nsec / NSEC_PER_USEC, > + .type = type, > + .code = code, > + .value = value, > + }; > + > udev->head = (udev->head + 1) % UINPUT_BUFFER_SIZE; > > wake_up_interruptible(&udev->waitq); > diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h > index f056b2a00d5c..9a61c28ed3ae 100644 > --- a/include/uapi/linux/input.h > +++ b/include/uapi/linux/input.h > @@ -34,6 +34,7 @@ struct input_event { > __kernel_ulong_t __sec; > #if defined(__sparc__) && defined(__arch64__) > unsigned int __usec; > + unsigned int __pad; > #else > __kernel_ulong_t __usec; > #endif > -- > 2.20.0 > -- Dmitry ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-12-13 22:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20191213204936.3643476-1-arnd@arndb.de>
2019-12-13 20:49 ` [PATCH v2 01/24] Input: input_event: fix struct padding on sparc64 Arnd Bergmann
2019-12-13 22:08 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).