regulator: bq257xx: Fix device node reference leak in bq257xx_reg_dt_parse_gpio()

In bq257xx_reg_dt_parse_gpio(), if fails to get subchild, it returns
without calling of_node_put(child), causing the device node reference
leak.

Fixes: 981dd162b635 ("regulator: bq257xx: Add bq257xx boost regulator driver")
Signed-off-by: Felix Gu <ustc.gu@gmail.com>
Link: https://patch.msgid.link/20260224-bq257-v1-1-8ebbc731c1c3@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/regulator/bq257xx-regulator.c b/drivers/regulator/bq257xx-regulator.c
index fc1ccede446882a1eda24ad5f9d2739cba89b9cb..dab8f1ab44503e57a906a44e9d1bb88d3cd46eaa 100644 (file)
--- a/drivers/regulator/bq257xx-regulator.c
+++ b/drivers/regulator/bq257xx-regulator.c
@@ -115,11 +115,10 @@ static void bq257xx_reg_dt_parse_gpio(struct platform_device *pdev)
                return;
 
        subchild = of_get_child_by_name(child, pdata->desc.of_match);
+       of_node_put(child);
        if (!subchild)
                return;
 
-       of_node_put(child);
-
        pdata->otg_en_gpio = devm_fwnode_gpiod_get_index(&pdev->dev,
                                                         of_fwnode_handle(subchild),
                                                         "enable", 0,