From: Jaco Coetzee Date: Thu, 17 Nov 2022 15:37:44 +0000 (+0100) Subject: nfp: add port from netdev validation for EEPROM access X-Git-Tag: baikal/mips/sdk5.8.2~1790 X-Git-Url: https://git.baikalelectronics.ru/?a=commitdiff_plain;h=aeae5045dcd4b58895f31fb4a4fdab4063c72066;p=kernel.git nfp: add port from netdev validation for EEPROM access [ Upstream commit 5d47f54ba55394dfdb6ce877811e0a18d7a0dfc2 ] Setting of the port flag `NFP_PORT_CHANGED`, introduced to ensure the correct reading of EEPROM data, causes a fatal kernel NULL pointer dereference in cases where the target netdev type cannot be determined. Add validation of port struct pointer before attempting to set the `NFP_PORT_CHANGED` flag. Return that operation is not supported if the netdev type cannot be determined. Fixes: 57b063516ab7 ("nfp: ethtool: fix the display error of `ethtool -m DEVNAME`") Signed-off-by: Jaco Coetzee Reviewed-by: Louis Peens Signed-off-by: Simon Horman Signed-off-by: Jakub Kicinski Signed-off-by: Sasha Levin --- diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c b/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c index 46d6988829ff1..ff88103571819 100644 --- a/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c @@ -1127,6 +1127,9 @@ nfp_port_get_module_info(struct net_device *netdev, u8 data; port = nfp_port_from_netdev(netdev); + if (!port) + return -EOPNOTSUPP; + /* update port state to get latest interface */ set_bit(NFP_PORT_CHANGED, &port->flags); eth_port = nfp_port_get_eth_port(port);