Commit ee662d44 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Mauro Carvalho Chehab

[media] media: am437x: Don't release OF node reference twice

The remote port reference is released both at the end of the OF graph
parsing loop, and in the error code path at the end of the function.
Those two calls will release the same reference, causing the reference
count to go negative.

Fix the problem by removing the second call.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: default avatarLad, Prabhakar <prabhakar.csengg@gmail.com>
Acked-by: default avatarBenoit Parrot <bparrot@ti.com>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 1cdf60c9
...@@ -2427,7 +2427,7 @@ static int vpfe_async_complete(struct v4l2_async_notifier *notifier) ...@@ -2427,7 +2427,7 @@ static int vpfe_async_complete(struct v4l2_async_notifier *notifier)
static struct vpfe_config * static struct vpfe_config *
vpfe_get_pdata(struct platform_device *pdev) vpfe_get_pdata(struct platform_device *pdev)
{ {
struct device_node *endpoint = NULL, *rem = NULL; struct device_node *endpoint = NULL;
struct v4l2_of_endpoint bus_cfg; struct v4l2_of_endpoint bus_cfg;
struct vpfe_subdev_info *sdinfo; struct vpfe_subdev_info *sdinfo;
struct vpfe_config *pdata; struct vpfe_config *pdata;
...@@ -2445,6 +2445,8 @@ vpfe_get_pdata(struct platform_device *pdev) ...@@ -2445,6 +2445,8 @@ vpfe_get_pdata(struct platform_device *pdev)
return NULL; return NULL;
for (i = 0; ; i++) { for (i = 0; ; i++) {
struct device_node *rem;
endpoint = of_graph_get_next_endpoint(pdev->dev.of_node, endpoint = of_graph_get_next_endpoint(pdev->dev.of_node,
endpoint); endpoint);
if (!endpoint) if (!endpoint)
...@@ -2515,7 +2517,6 @@ vpfe_get_pdata(struct platform_device *pdev) ...@@ -2515,7 +2517,6 @@ vpfe_get_pdata(struct platform_device *pdev)
done: done:
of_node_put(endpoint); of_node_put(endpoint);
of_node_put(rem);
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment