github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-06-06 21:45:45 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

media: xilinx: vipp: Fix refcount leak in xvip_graph_dma_init

Browse Source 
[ Upstream commit 1c78f19c3a ]

of_get_child_by_name() returns a node pointer with refcount
incremented, we should use of_node_put() on it when not need anymore.
Add missing of_node_put() to avoid refcount leak.

Fixes: df3305156f ("[media] v4l: xilinx: Add Xilinx Video IP core")
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Miaoqian Lin 2022-06-01 06:25:14 +02:00 committed by Greg Kroah-Hartman
parent 40aa0999a3
commit 2630cc8832
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/media/platform/xilinx/xilinx-vipp.c
View File

@ -472,7 +472,7 @@ static int xvip_graph_dma_init(struct xvip_composite_device *xdev)
{ {
struct device_node *ports; struct device_node *ports;
struct device_node *port; struct device_node *port;
int ret; int ret = 0;
ports = of_get_child_by_name(xdev->dev->of_node, "ports"); ports = of_get_child_by_name(xdev->dev->of_node, "ports");
if (ports == NULL) { if (ports == NULL) {
@ -482,13 +482,14 @@ static int xvip_graph_dma_init(struct xvip_composite_device *xdev)
for_each_child_of_node(ports, port) { for_each_child_of_node(ports, port) {
ret = xvip_graph_dma_init_one(xdev, port); ret = xvip_graph_dma_init_one(xdev, port);
if (ret < 0) { if (ret) {
of_node_put(port); of_node_put(port);
return ret; break;
} }
} }
return 0; of_node_put(ports);
return ret;
} }
static void xvip_graph_cleanup(struct xvip_composite_device *xdev) static void xvip_graph_cleanup(struct xvip_composite_device *xdev)