* [PATCH v3] usb: usbip: fix a refcount leak in stub_probe()
@ 2022-04-12 2:02 Hangyu Hua
2022-04-12 14:44 ` Shuah Khan
0 siblings, 1 reply; 2+ messages in thread
From: Hangyu Hua @ 2022-04-12 2:02 UTC (permalink / raw)
To: valentina.manea.m, shuah, gregkh, khoroshilov, skhan
Cc: linux-usb, linux-kernel, Hangyu Hua
usb_get_dev() is called in stub_device_alloc(). When stub_probe() fails
after that, usb_put_dev() needs to be called to release the reference.
Fix this by moving usb_put_dev() to sdev_free error path handling.
Find this by code review.
Fixes: 3ff67445750a ("usbip: fix error handling in stub_probe()")
Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
Reviewed-by: Shuah Khan <skhan@linuxfoundation.org>
---
v2: add more description of this patch.
v3: add how to find the problem.
drivers/usb/usbip/stub_dev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/usbip/stub_dev.c b/drivers/usb/usbip/stub_dev.c
index d8d3892e5a69..3c6d452e3bf4 100644
--- a/drivers/usb/usbip/stub_dev.c
+++ b/drivers/usb/usbip/stub_dev.c
@@ -393,7 +393,6 @@ static int stub_probe(struct usb_device *udev)
err_port:
dev_set_drvdata(&udev->dev, NULL);
- usb_put_dev(udev);
/* we already have busid_priv, just lock busid_lock */
spin_lock(&busid_priv->busid_lock);
@@ -408,6 +407,7 @@ static int stub_probe(struct usb_device *udev)
put_busid_priv(busid_priv);
sdev_free:
+ usb_put_dev(udev);
stub_device_free(sdev);
return rc;
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v3] usb: usbip: fix a refcount leak in stub_probe()
2022-04-12 2:02 [PATCH v3] usb: usbip: fix a refcount leak in stub_probe() Hangyu Hua
@ 2022-04-12 14:44 ` Shuah Khan
0 siblings, 0 replies; 2+ messages in thread
From: Shuah Khan @ 2022-04-12 14:44 UTC (permalink / raw)
To: Hangyu Hua, valentina.manea.m, shuah, gregkh, khoroshilov
Cc: linux-usb, linux-kernel, Shuah Khan
On 4/11/22 8:02 PM, Hangyu Hua wrote:
> usb_get_dev() is called in stub_device_alloc(). When stub_probe() fails
> after that, usb_put_dev() needs to be called to release the reference.
>
> Fix this by moving usb_put_dev() to sdev_free error path handling.
>
> Find this by code review.
>
> Fixes: 3ff67445750a ("usbip: fix error handling in stub_probe()")
> Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
> Reviewed-by: Shuah Khan <skhan@linuxfoundation.org>
> ---
>
> v2: add more description of this patch.
>
> v3: add how to find the problem.
>
Thank you.
Greg, Please pick this patch up.
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-12 14:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-12 2:02 [PATCH v3] usb: usbip: fix a refcount leak in stub_probe() Hangyu Hua
2022-04-12 14:44 ` Shuah Khan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).