From: Johan Hovold Date: Wed, 17 Mar 2010 22:00:43 +0000 (+0100) Subject: USB: serial: fix missing locking on fifo in write callback X-Git-Tag: v2.6.35-rc1~471^2~194 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=50dbb8528757b1977efd5d270ed9d262cbbef87d;p=platform%2Fkernel%2Flinux-exynos.git USB: serial: fix missing locking on fifo in write callback On errors the fifo was reset without any locking. This could race with write which do kfifo_put and perhaps also chars_in_buffer and write_room. Every other access to the fifo is protected using the port lock so better add it to the error path as well. Signed-off-by: Johan Hovold Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c index 8f78d7b..2a3196a 100644 --- a/drivers/usb/serial/generic.c +++ b/drivers/usb/serial/generic.c @@ -519,10 +519,13 @@ void usb_serial_generic_write_bulk_callback(struct urb *urb) port->write_urb_busy = 0; spin_unlock_irqrestore(&port->lock, flags); - if (status) + if (status) { + spin_lock_irqsave(&port->lock, flags); kfifo_reset_out(&port->write_fifo); - else + spin_unlock_irqrestore(&port->lock, flags); + } else { usb_serial_generic_write_start(port); + } } if (status)