Commit 76719868 authored by Alexey Khoroshilov's avatar Alexey Khoroshilov Committed by Marc Kleine-Budde

can: usb_8dev: fix urb leak on failure path in usb_8dev_start()

If usb_8dev_start() fails to submit urb,
it unanchors the urb but forgets to free it.

Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: default avatarAlexey Khoroshilov <khoroshilov@ispras.ru>
Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent 233a26e8
...@@ -779,6 +779,7 @@ static int usb_8dev_start(struct usb_8dev_priv *priv) ...@@ -779,6 +779,7 @@ static int usb_8dev_start(struct usb_8dev_priv *priv)
usb_unanchor_urb(urb); usb_unanchor_urb(urb);
usb_free_coherent(priv->udev, RX_BUFFER_SIZE, buf, usb_free_coherent(priv->udev, RX_BUFFER_SIZE, buf,
urb->transfer_dma); urb->transfer_dma);
usb_free_urb(urb);
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment