lpc_eth: kernel BUG on remove
authorBruno Carneiro da Cunha <brunocarneirodacunha@usp.br>
Thu, 5 Dec 2019 20:16:26 +0000 (17:16 -0300)
committerDavid S. Miller <davem@davemloft.net>
Sat, 7 Dec 2019 04:51:19 +0000 (20:51 -0800)
commit04aa1bc42e4d31a7c58f38eefc323ac395517635
tree7ea4fe671733582122197606e02c78fa4ba04d06
parent9424e2e7ad93ffffa88f882c9bc5023570904b55
lpc_eth: kernel BUG on remove

We may have found a bug in the nxp/lpc_eth.c driver. The function
platform_set_drvdata() is called twice, the second time it is called,
in lpc_mii_init(), it overwrites the struct net_device which should be
at pdev->dev->driver_data with pldat->mii_bus. When trying to remove
the driver, in lpc_eth_drv_remove(), platform_get_drvdata() will
return the pldat->mii_bus pointer and try to use it as a struct
net_device pointer. This causes unregister_netdev to segfault and
generate a kernel BUG. Is this reproducible?

Signed-off-by: Daniel Martinez <linux@danielsmartinez.com>
Signed-off-by: Bruno Carneiro da Cunha <brunocarneirodacunha@usp.br>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/nxp/lpc_eth.c