]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
USB: serial: io_edgeport: fix possible sleep-in-atomic
authorJia-Ju Bai <baijiaju1990@gmail.com>
Wed, 13 Dec 2017 12:34:36 +0000 (20:34 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 7 Feb 2018 19:07:57 +0000 (11:07 -0800)
commit464bab18479200df3582da84f4cf6861186e7a9a
tree224c10a955df5f4db8ad1ae6d00f134226eb5e91
parentd1cfbc6e33647497f08b3d7bcceb400f49802bdb
USB: serial: io_edgeport: fix possible sleep-in-atomic

commit c7b8f77872c73f69a16528a9eb87afefcccdc18b upstream.

According to drivers/usb/serial/io_edgeport.c, the driver may sleep
under a spinlock.
The function call path is:
edge_bulk_in_callback (acquire the spinlock)
   process_rcvd_data
     process_rcvd_status
       change_port_settings
         send_iosp_ext_cmd
           write_cmd_usb
             usb_kill_urb --> may sleep

To fix it, the redundant usb_kill_urb() is removed from the error path
after usb_submit_urb() fails.

This possible bug is found by my static analysis tool (DSAC) and checked
by my code review.

Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/io_edgeport.c