[PATCH 04/44] coresight: platform: Fix leaking device reference

From: Mathieu Poirier
Date: Thu Sep 20 2018 - 15:22:18 EST


From: Suzuki K Poulose <suzuki.poulose@xxxxxxx>

We don't drop the reference on the remote device while parsing the
connection, held by bus_find_device(). Fix this by duplicating the
device name and dropping the reference.

Cc: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
Cc: Kim Phillips <kim.phillips@xxxxxxx>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
Signed-off-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
---
drivers/hwtracing/coresight/of_coresight.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/hwtracing/coresight/of_coresight.c b/drivers/hwtracing/coresight/of_coresight.c
index 28d3aef1660b..4b279f8fea0c 100644
--- a/drivers/hwtracing/coresight/of_coresight.c
+++ b/drivers/hwtracing/coresight/of_coresight.c
@@ -162,7 +162,9 @@ static int of_coresight_parse_endpoint(struct device *dev,
}

pdata->outports[i] = endpoint.port;
- pdata->child_names[i] = dev_name(rdev);
+ pdata->child_names[i] = devm_kstrdup(dev,
+ dev_name(rdev),
+ GFP_KERNEL);
pdata->child_ports[i] = rendpoint.id;
/* Connection record updated */
ret = 1;
@@ -172,6 +174,8 @@ static int of_coresight_parse_endpoint(struct device *dev,
of_node_put(rparent);
if (rport)
of_node_put(rport);
+ if (rdev)
+ put_device(rdev);

return ret;
}
--
2.7.4