OSDN Git Service

spi: cadence-quadspi: add missing of_node_put
authorJunlin Yang <yangjunlin@yulong.com>
Mon, 15 Feb 2021 11:04:25 +0000 (19:04 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 10 Mar 2021 12:46:54 +0000 (12:46 +0000)
Fix OF node leaks by calling of_node_put in
for_each_available_child_of_node when the cycle returns.

Generated by: scripts/coccinelle/iterators/for_each_child.cocci

Signed-off-by: Junlin Yang <yangjunlin@yulong.com>
Link: https://lore.kernel.org/r/20210215110425.1012-1-angkery@163.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-cadence-quadspi.c

index 442cc7c..14a7120 100644 (file)
@@ -1389,11 +1389,13 @@ static int cqspi_setup_flash(struct cqspi_st *cqspi)
                ret = of_property_read_u32(np, "reg", &cs);
                if (ret) {
                        dev_err(dev, "Couldn't determine chip select.\n");
+                       of_node_put(np);
                        return ret;
                }
 
                if (cs >= CQSPI_MAX_CHIPSELECT) {
                        dev_err(dev, "Chip select %d out of range.\n", cs);
+                       of_node_put(np);
                        return -EINVAL;
                }
 
@@ -1402,8 +1404,10 @@ static int cqspi_setup_flash(struct cqspi_st *cqspi)
                f_pdata->cs = cs;
 
                ret = cqspi_of_get_flash_pdata(pdev, f_pdata, np);
-               if (ret)
+               if (ret) {
+                       of_node_put(np);
                        return ret;
+               }
        }
 
        return 0;