Thread (10 messages) flat view 10 messages, 2 authors, 2021-03-23
STALE1964d

Revision v2 of 2 in this series.

Revisions (2)
  1. v1 [diff vs current]
  2. v2 current

[PATCH v2 0/8] USB: cdc-acm: probe fixes

From: Johan Hovold <johan@kernel.org>
Date: 2021-03-22 15:53:52
Also in: lkml

This series fixes a couple of bugs in the probe errors paths and does
some clean up in preparation for adding the missing error handling when
claiming the data interface.

The first two should probably go into 5.12-rc, while the rest could be
held off for 5.13 if preferred.

Johan


Changes in v2
 - clarify that the driver already had a check for a bound data
   interface and remove the now redundant check (Oliver) (7/8)
 - add Oliver's ack to patches 1 through 6
 - add new to avoid logging a successful probe message on late errors
   (8/8)

Johan Hovold (8):
  USB: cdc-acm: fix double free on probe failure
  USB: cdc-acm: fix use-after-free after probe failure
  USB: cdc-acm: drop redundant driver-data assignment
  USB: cdc-acm: drop redundant driver-data reset
  USB: cdc-acm: clean up probe error labels
  USB: cdc-acm: use negation for NULL checks
  USB: cdc-acm: always claim data interface
  USB: cdc-acm: do not log successful probe on later errors

 drivers/usb/class/cdc-acm.c | 65 +++++++++++++++++++------------------
 1 file changed, 33 insertions(+), 32 deletions(-)

-- 
2.26.3
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help