net: netcp: drop kfree for memory allocated with devm_kzalloc
authorWei Yongjun <weiyongjun1@huawei.com>
Sat, 22 Oct 2016 12:32:04 +0000 (12:32 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 26 Oct 2016 21:21:59 +0000 (17:21 -0400)
It's not necessary to free memory allocated with devm_kzalloc in the
remove path and using kfree leads to a double free.

Fixes: 84640e27f230 ("net: netcp: Add Keystone NetCP core ethernet
driver")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ti/netcp_core.c

index 11609d5467fcb4593728f597414711935b9130b8..2fb5b6dd5231a4ccde1c3154eef98f96f2a6915d 100644 (file)
@@ -2058,7 +2058,6 @@ static void netcp_delete_interface(struct netcp_device *netcp_device,
                if (module->release)
                        module->release(intf_modpriv->module_priv);
                list_del(&intf_modpriv->intf_list);
-               kfree(intf_modpriv);
        }
        WARN(!list_empty(&netcp->module_head), "%s interface module list is not empty!\n",
             ndev->name);
@@ -2153,7 +2152,6 @@ static int netcp_remove(struct platform_device *pdev)
                dev_dbg(&pdev->dev, "Removing module \"%s\"\n", module->name);
                module->remove(netcp_device, inst_modpriv->module_priv);
                list_del(&inst_modpriv->inst_list);
-               kfree(inst_modpriv);
        }
 
        /* now that all modules are removed, clean up the interfaces */