net: ethernet: sunplus: add missing of_node_put() in spl2sw_mdio_init()
authorYang Yingliang <yangyingliang@huawei.com>
Wed, 18 May 2022 02:08:12 +0000 (10:08 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 May 2022 13:08:49 +0000 (14:08 +0100)
of_get_child_by_name() returns device node pointer with refcount
incremented. The refcount should be decremented before returning
from spl2sw_mdio_init().

Fixes: fd3040b9394c ("net: ethernet: Add driver for Sunplus SP7021")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Wells Lu <wellslutw@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sunplus/spl2sw_mdio.c

index 139ac8f..733ae17 100644 (file)
@@ -97,8 +97,10 @@ u32 spl2sw_mdio_init(struct spl2sw_common *comm)
 
        /* Allocate and register mdio bus. */
        mii_bus = devm_mdiobus_alloc(&comm->pdev->dev);
-       if (!mii_bus)
-               return -ENOMEM;
+       if (!mii_bus) {
+               ret = -ENOMEM;
+               goto out;
+       }
 
        mii_bus->name = "sunplus_mii_bus";
        mii_bus->parent = &comm->pdev->dev;
@@ -110,10 +112,13 @@ u32 spl2sw_mdio_init(struct spl2sw_common *comm)
        ret = of_mdiobus_register(mii_bus, mdio_np);
        if (ret) {
                dev_err(&comm->pdev->dev, "Failed to register mdiobus!\n");
-               return ret;
+               goto out;
        }
 
        comm->mii_bus = mii_bus;
+
+out:
+       of_node_put(mdio_np);
        return ret;
 }