diff options
| author | Edward Cree <ecree@solarflare.com> | 2020-08-18 14:44:18 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-08-18 21:49:12 +0200 |
| commit | 788f920a0f137baa4dbc1efdd5039c4a0a01b8d7 (patch) | |
| tree | 90ea280adcddc85db7f3ead28d589aa16cda6425 /net | |
| parent | sfc: take correct lock in ef100_reset() (diff) | |
| download | linux-788f920a0f137baa4dbc1efdd5039c4a0a01b8d7.tar.xz linux-788f920a0f137baa4dbc1efdd5039c4a0a01b8d7.zip | |
sfc: null out channel->rps_flow_id after freeing it
If an ef100_net_open() fails, ef100_net_stop() may be called without
channel->rps_flow_id having been written; thus it may hold the address
freed by a previous ef100_net_stop()'s call to efx_remove_filters().
This then causes a double-free when efx_remove_filters() is called
again, leading to a panic.
To prevent this, after freeing it, overwrite it with NULL.
Fixes: a9dc3d5612ce ("sfc_ef100: RX filter table management and related gubbins")
Signed-off-by: Edward Cree <ecree@solarflare.com>
Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions