Re: [PATCH net] net: hibmcge: set netdev_ops before hbg_init() to fix NULL pointer dereference

[Daniel Machon]

> The driver assigns netdev_ops after hbg_init(), but hbg_init() calls
> hbg_mdio_init() -> hbg_phy_connect() -> phy_connect_direct() ->
> phy_attach_direct() -> phy_link_topo_add_phy(), which invokes
> netdev_need_ops_lock(). With CONFIG_NET_SHAPER enabled, this function
> dereferences dev->netdev_ops->net_shaper_ops. Since netdev_ops is still
> NULL at this point, a NULL pointer dereference occurs.
> 
> Move the netdev_ops assignment before hbg_init() so that
> netdev_need_ops_lock() can safely access netdev_ops during PHY
> attachment.

Reviewed-by: Daniel Machon <daniel.machon@xxxxxxxxxxxxx>