Skip to content

serdev: receive_buf and locking fixes

Johan Hovold linux-serial lkml stable 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 6 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8serdev: ttyport: add missing receive_buf sanity checksdrivers/tty/serdev/serdev-ttyport.c+12 -10 replies
2/8serdev: fix receive_buf return value when no callbackinclude/linux/serdev.h+1 -10 replies
3/8serdev: document driver callbacksinclude/linux/serdev.h+4 -20 replies
4/8serdev: ttyport: fix NULL-deref on hangupdrivers/tty/serdev/serdev-ttyport.c+9 -20 replies
5/8serdev: ttyport: fix tty locking in closedrivers/tty/serdev/serdev-ttyport.c+2 -00 replies
6/8serdev: ttyport: release tty lock sooner on opendrivers/tty/serdev/serdev-ttyport.c+2 -10 replies
7/8serdev: ttyport: ignore carrier detect to avoid hangupsdrivers/tty/serdev/serdev-ttyport.c+2 -00 replies
8/8serdev: ttyport: do not used keyed wakeup in write_wakeupdrivers/tty/serdev/serdev-ttyport.c+2 -16 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/8] serdev: receive_buf and locking fixes

This series addresses a few issues with the serdev code, including potential information leaks due to missing sanity checks in the receive path, a NULL-deref in write_wakeup() due to missing reference handling, and missing tty locking in close().

Johan

Johan Hovold (8):
serdev: ttyport: add missing receive_buf sanity checks
serdev: fix receive_buf return value when no callback
serdev: document driver callbacks
serdev: ttyport: fix NULL-deref on hangup
serdev: ttyport: fix tty locking in close
serdev: ttyport: release tty lock sooner on open
serdev: ttyport: ignore carrier detect to avoid hangups
serdev: ttyport: do not used keyed wakeup in write_wakeup

 drivers/tty/serdev/serdev-ttyport.c | 32 ++++++++++++++++++++++++++++----
 include/linux/serdev.h              |  8 +++++---
 2 files changed, 33 insertions(+), 7 deletions(-)
Signature · 1 line
2.15.0