From: Hui Peng <benquike@gmail.com>
To: gregkh@linuxfoundation.org, jirislaby@kernel.org,
john.ogness@linutronix.de, ilpo.jarvinen@linux.intel.com
Cc: andriy.shevchenko@linux.intel.com, linux-serial@vger.kernel.org,
linux-kernel@vger.kernel.org, stable@vger.kernel.org,
Hui Peng <benquike@gmail.com>
Subject: [PATCH v4 0/3] serial: fix IRQ chain race, baud fallback, and uartclk overflow
Date: Thu, 24 Sep 2026 06:04:43 +0000 [thread overview]
Message-ID: <20260924060511.2364717-1-benquike@gmail.com> (raw)
This series fixes three bugs in the 8250 and serial core drivers:
1. Hold hash_mutex across IRQ chain lookup, linking, and error-path
unlinking in serial_link_irq_chain() to prevent a use-after-free race
with concurrent serial_unlink_irq_chain().
2. Allow a third loop iteration (try < 3) in uart_get_baud_rate() so that
when both the requested and old baud rates are out of range and try == 1
clips the rate into termios, the clipped baud rate is evaluated and
returned on try == 2 instead of returning 0 and causing a divide-by-zero
in uart_get_divisor().
3. Reject baud_base > UINT_MAX / 16 in uart_set_info() alongside the
uartclk == 0 check so that multiplying baud_base by 16 cannot wrap
around in 32-bit unsigned arithmetic to a small non-zero uartclk.
Changes in v3:
- Patch 2/3: Collect Reviewed-by from Ilpo Järvinen.
- Patch 3/3: Add missing #include <linux/limits.h> for UINT_MAX and move
the new_info->baud_base > UINT_MAX / 16 check to the uartclk == 0 check,
as suggested by Ilpo Järvinen.
Changes in v2:
- Split the 8250_core.c and serial_core.c fixes into a 3-patch series,
add Cc: stable@vger.kernel.org, and document how each patch was tested in
QEMU, as requested by Greg Kroah-Hartman and John Ogness.
Hui Peng (3):
serial: 8250: hold hash_mutex across IRQ chain linking in
serial_link_irq_chain()
serial: core: allow third iteration in uart_get_baud_rate() fallback
serial: core: reject baud_base values that overflow port->uartclk in
uart_set_info()
drivers/tty/serial/8250/8250_core.c | 4 ++--
drivers/tty/serial/serial_core.c | 5 +++--
2 files changed, 5 insertions(+), 4 deletions(-)
--
2.49.0
next reply other threads:[~2026-09-24 6:05 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-24 6:04 Hui Peng [this message]
2026-09-24 6:04 ` [PATCH v4 1/3] serial: 8250: hold hash_mutex across IRQ chain linking in serial_link_irq_chain() Hui Peng
2026-09-24 6:11 ` sashiko-bot
2026-09-24 6:13 ` sashiko-bot
2026-09-24 6:04 ` [PATCH v4 2/3] serial: core: allow third iteration in uart_get_baud_rate() fallback Hui Peng
2026-09-24 6:13 ` sashiko-bot
2026-09-24 12:29 ` Andy Shevchenko
2026-09-24 6:42 ` [PATCH v4 0/3] serial: fix IRQ chain race, baud fallback, and uartclk overflow Greg KH
2026-09-24 6:54 ` [PATCH v4 3/3] serial: core: reject baud_base values that overflow port->uartclk in uart_set_info() Hui Peng
2026-09-24 11:40 ` Ilpo Järvinen
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=20260924060511.2364717-1-benquike@gmail.com \
--to=benquike@gmail.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=jirislaby@kernel.org \
--cc=john.ogness@linutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=stable@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.