From: Enrik Berkhan <Enrik.Berkhan@inka.de>
To: linux-input@vger.kernel.org
Cc: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
Rishi Gupta <gupt21@gmail.com>,
Enrik Berkhan <Enrik.Berkhan@inka.de>
Subject: [RESEND PATCH v1 0/4] Fixes for the mcp2221 HID-to-I2C-bridge driver
Date: Mon, 26 Sep 2022 22:22:35 +0200 [thread overview]
Message-ID: <20220926202239.16379-1-Enrik.Berkhan@inka.de> (raw)
In this patch series are fixes for issues I found during recent tests of
an MCP2221 board.
- you can confuse the kernel driver when using the chip from user mode
via /dev/hidrawX, typically leading to a NULL pointer dereference in
the driver's HID raw event handler
- the driver needs > 15s to initialize because the HID raw handling is
not enabled during initialization of the GPIO part
- data shared with the bottom half code is not protected from concurrent
access
- the rxbuf pointer can become invalid or even stale if the device would
send unsolicited reports
Enrik Berkhan (4):
HID: mcp2221: don't connect hidraw
HID: mcp2221: enable HID I/O during GPIO probe
HID: mcp2221: protect shared data with spin lock
HID: mcp2221: avoid stale rxbuf pointer
drivers/hid/hid-mcp2221.c | 114 ++++++++++++++++++++++++++++++++------
1 file changed, 97 insertions(+), 17 deletions(-)
---
Resend because I had a typo in the linux-input mailing list address.
Also adding linux-kernel to increase visibility.
--
2.34.1
next reply other threads:[~2022-09-26 21:20 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-26 20:22 Enrik Berkhan [this message]
2022-09-26 20:22 ` [PATCH v1 1/4] HID: mcp2221: don't connect hidraw Enrik Berkhan
2022-09-26 20:22 ` [PATCH v1 2/4] HID: mcp2221: enable HID I/O during GPIO probe Enrik Berkhan
2022-09-26 20:22 ` [PATCH v1 3/4] HID: mcp2221: protect shared data with spin lock Enrik Berkhan
2022-09-26 20:22 ` [PATCH v1 4/4] HID: mcp2221: avoid stale rxbuf pointer Enrik Berkhan
2022-11-03 22:27 ` [PATCH v2 0/3] Fixes for the mcp2221 HID-to-I2C-bridge driver Enrik Berkhan
2022-11-03 22:27 ` [PATCH v2 1/3] HID: mcp2221: don't connect hidraw Enrik Berkhan
2022-12-19 14:11 ` Benjamin Tissoires
2022-11-03 22:27 ` [PATCH v2 2/3] HID: mcp2221: enable HID I/O during GPIO probe Enrik Berkhan
2022-11-03 22:27 ` [PATCH v2 3/3] HID: mcp2221: avoid stale rxbuf pointer Enrik Berkhan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220926202239.16379-1-Enrik.Berkhan@inka.de \
--to=enrik.berkhan@inka.de \
--cc=gupt21@gmail.com \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.