projects
/
GitHub
/
LineageOS
/
android_kernel_motorola_exynos9610.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
372d735
)
regulator: of: Fix refcount leak bug in of_get_regulation_constraints()
author
Liang He
<windhl@126.com>
Fri, 15 Jul 2022 11:10:27 +0000
(19:10 +0800)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Thu, 25 Aug 2022 09:11:16 +0000
(11:11 +0200)
[ Upstream commit
66efb665cd5ad69b27dca8571bf89fc6b9c628a4
]
We should call the of_node_put() for the reference returned by
of_get_child_by_name() which has increased the refcount.
Fixes:
40e20d68bb3f
("regulator: of: Add support for parsing regulator_state for suspend state")
Signed-off-by: Liang He <windhl@126.com>
Link:
https://lore.kernel.org/r/20220715111027.391032-1-windhl@126.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/regulator/of_regulator.c
patch
|
blob
|
blame
|
history
diff --git
a/drivers/regulator/of_regulator.c
b/drivers/regulator/of_regulator.c
index a3bf7c993723acdaab9160b22be58a7e5d52b37a..f82b522bffa7d603d1b5c82cfc581a63fa364e15 100644
(file)
--- a/
drivers/regulator/of_regulator.c
+++ b/
drivers/regulator/of_regulator.c
@@
-158,8
+158,12
@@
static void of_get_regulation_constraints(struct device_node *np,
}
suspend_np = of_get_child_by_name(np, regulator_states[i]);
- if (!suspend_np
|| !suspend_state
)
+ if (!suspend_np)
continue;
+ if (!suspend_state) {
+ of_node_put(suspend_np);
+ continue;
+ }
if (!of_property_read_u32(suspend_np, "regulator-mode",
&pval)) {