Skip to content

Commit de644a4

Browse files
Dan Carpentergregkh
authored andcommitted
usb: musb: da8xx: fix a resource leak in probe()
Call usb_phy_generic_unregister() if of_platform_populate() fails. Fixes: d6299b6 ("usb: musb: Add support of CPPI 4.1 DMA controller to DA8xx") Cc: stable <[email protected]> Signed-off-by: Dan Carpenter <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent 9e3caa9 commit de644a4

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

drivers/usb/musb/da8xx.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -556,7 +556,7 @@ static int da8xx_probe(struct platform_device *pdev)
556556
ret = of_platform_populate(pdev->dev.of_node, NULL,
557557
da8xx_auxdata_lookup, &pdev->dev);
558558
if (ret)
559-
return ret;
559+
goto err_unregister_phy;
560560

561561
pinfo = da8xx_dev_info;
562562
pinfo.parent = &pdev->dev;
@@ -571,9 +571,13 @@ static int da8xx_probe(struct platform_device *pdev)
571571
ret = PTR_ERR_OR_ZERO(glue->musb);
572572
if (ret) {
573573
dev_err(&pdev->dev, "failed to register musb device: %d\n", ret);
574-
usb_phy_generic_unregister(glue->usb_phy);
574+
goto err_unregister_phy;
575575
}
576576

577+
return 0;
578+
579+
err_unregister_phy:
580+
usb_phy_generic_unregister(glue->usb_phy);
577581
return ret;
578582
}
579583

0 commit comments

Comments
 (0)