Skip to contentSkip to conversation

patch "USB: yurex: fix control-URB timeout handling" added to usb-linus

PATCH [email protected] 1 participant 1 message Dec 28, 2020

Subsystems (2)
lore
Share & tools
lore

[email protected]

author maintainer Applied
#0 lore raw

This is a note to let you know that I've just added the patch titled

USB: yurex: fix control-URB timeout handling

to my usb git tree which can be found at
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git
in the usb-linus branch.

The patch will show up in the next release of the linux-next tree (usually sometime within the next 24 hours during the week.)

The patch will hopefully also be merged in Linus's tree for the next -rc kernel release.

If you have any questions about this process, please let me know.

From 372c93131998c0622304bed118322d2a04489e63 Mon Sep 17 00:00:00 2001
From: Johan Hovold <[email protected]>
Date: Mon, 14 Dec 2020 11:30:53 +0100
Subject: USB: yurex: fix control-URB timeout handling

Make sure to always cancel the control URB in write() so that it can be reused after a timeout or spurious CMD_ACK.

Currently any further write requests after a timeout would fail after triggering a WARN() in usb_submit_urb() when attempting to submit the already active URB.

Reported-by [email protected]
Fixes 6bc235a2e24a ("USB: add driver for Meywa-Denki & Kayac YUREX")
Cc stable <[email protected]> # 2.6.37
---
 drivers/usb/misc/yurex.c | 3 +++
 1 file changed, 3 insertions(+)
drivers/usb/misc/yurex.c +3 -0
index 73ebfa6e9715..c640f98d20c5 100644
@@ -496,6 +496,9 @@ static ssize_t yurex_write(struct file *file, const char __user *user_buffer,
 		timeout = schedule_timeout(YUREX_WRITE_TIMEOUT);
 	finish_wait(&dev->waitq, &wait);
 
+	/* make sure URB is idle after timeout or (spurious) CMD_ACK */
+	usb_kill_urb(dev->cntl_urb);
+
 	mutex_unlock(&dev->io_mutex);
 
 	if (retval < 0) {
Signature · 1 line
2.29.2