diff options
| author | Martin Habets <[email protected]> | 2023-06-28 13:32:20 +0100 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2023-06-29 11:10:27 -0700 |
| commit | 915057ae79692d47f9fb3504785855be49abaea4 (patch) | |
| tree | 5bf58fa9f005ed4da80e3605a1c08eb7644f0a1e | |
| parent | 56f66ce1fd41c8eb063b550581d664af1e576b55 (diff) | |
sfc: support for devlink port requires MAE access
On systems without MAE permission efx->mae is not initialised,
and trying to lookup an mport results in a NULL pointer
dereference.
Fixes: 25414b2a64ae ("sfc: add devlink port support for ef100")
Signed-off-by: Martin Habets <[email protected]>
Reviewed-by: Jacob Keller <[email protected]>
Signed-off-by: Jakub Kicinski <[email protected]>
| -rw-r--r-- | drivers/net/ethernet/sfc/efx_devlink.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/net/ethernet/sfc/efx_devlink.c b/drivers/net/ethernet/sfc/efx_devlink.c index b82dad50a5b1..3cd750820fdd 100644 --- a/drivers/net/ethernet/sfc/efx_devlink.c +++ b/drivers/net/ethernet/sfc/efx_devlink.c @@ -626,6 +626,9 @@ static struct devlink_port *ef100_set_devlink_port(struct efx_nic *efx, u32 idx) u32 id; int rc; + if (!efx->mae) + return NULL; + if (efx_mae_lookup_mport(efx, idx, &id)) { /* This should not happen. */ if (idx == MAE_MPORT_DESC_VF_IDX_NULL) |