Oliver Neukum 7a31d4
From fa0ef93868a6062babe1144df2807a8b1d4924d2 Mon Sep 17 00:00:00 2001
Oliver Neukum 7a31d4
From: Miaoqian Lin <linmq006@gmail.com>
Oliver Neukum 7a31d4
Date: Fri, 31 Dec 2021 11:36:41 +0000
Oliver Neukum 7a31d4
Subject: [PATCH] usb: dwc3: dwc3-qcom: Add missing platform_device_put() in
Oliver Neukum 7a31d4
 dwc3_qcom_acpi_register_core
Oliver Neukum 7a31d4
Git-commit: fa0ef93868a6062babe1144df2807a8b1d4924d2
Oliver Neukum 7a31d4
Patch-mainline: v5.17-rc1
Oliver Neukum 7a31d4
References: bsc#1208741 CVE-2023-22995
Oliver Neukum 7a31d4
Oliver Neukum 7a31d4
Add the missing platform_device_put() before return from
Oliver Neukum 7a31d4
dwc3_qcom_acpi_register_core in the error handling case.
Oliver Neukum 7a31d4
Oliver Neukum 7a31d4
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
Oliver Neukum 7a31d4
Link: https://lore.kernel.org/r/20211231113641.31474-1-linmq006@gmail.com
Oliver Neukum 7a31d4
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Oliver Neukum 7a31d4
Signed-off-by: Oliver Neukum <oneukum@suse.com>
Oliver Neukum 7a31d4
---
Oliver Neukum 7a31d4
 drivers/usb/dwc3/dwc3-qcom.c |   11 +++++++++--
Oliver Neukum 7a31d4
 1 file changed, 9 insertions(+), 2 deletions(-)
Oliver Neukum 7a31d4
Oliver Neukum 7a31d4
--- a/drivers/usb/dwc3/dwc3-qcom.c
Oliver Neukum 7a31d4
+++ b/drivers/usb/dwc3/dwc3-qcom.c
Oliver Neukum 7a31d4
@@ -474,8 +474,10 @@ static int dwc3_qcom_acpi_register_core(
Oliver Neukum 7a31d4
 	qcom->dwc3->dev.coherent_dma_mask = dev->coherent_dma_mask;
Oliver Neukum 7a31d4
 
Oliver Neukum 7a31d4
 	child_res = kcalloc(2, sizeof(*child_res), GFP_KERNEL);
Oliver Neukum 7a31d4
-	if (!child_res)
Oliver Neukum 7a31d4
+	if (!child_res) {
Oliver Neukum 7a31d4
+		platform_device_put(qcom->dwc3);
Oliver Neukum 7a31d4
 		return -ENOMEM;
Oliver Neukum 7a31d4
+	}
Oliver Neukum 7a31d4
 
Oliver Neukum 7a31d4
 	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
Oliver Neukum 7a31d4
 	if (!res) {
Oliver Neukum 7a31d4
@@ -507,10 +509,15 @@ static int dwc3_qcom_acpi_register_core(
Oliver Neukum 7a31d4
 	}
Oliver Neukum 7a31d4
 
Oliver Neukum 7a31d4
 	ret = platform_device_add(qcom->dwc3);
Oliver Neukum 7a31d4
-	if (ret)
Oliver Neukum 7a31d4
+	if (ret) {
Oliver Neukum 7a31d4
 		dev_err(&pdev->dev, "failed to add device\n");
Oliver Neukum 7a31d4
+		goto out;
Oliver Neukum 7a31d4
+	}
Oliver Neukum 7a31d4
+	kfree(child_res);
Oliver Neukum 7a31d4
+	return 0;
Oliver Neukum 7a31d4
 
Oliver Neukum 7a31d4
 out:
Oliver Neukum 7a31d4
+	platform_device_put(qcom->dwc3);
Oliver Neukum 7a31d4
 	kfree(child_res);
Oliver Neukum 7a31d4
 	return ret;
Oliver Neukum 7a31d4
 }