diff options
author | Luca Pesce <luca.pesce@vimar.com> | 2020-12-24 11:51:59 +0100 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2021-01-14 19:32:32 +0200 |
commit | e862a3e4088070de352fdafe9bd9e3ae0a95a33c (patch) | |
tree | 9c521820732f9ae840a6e36ec903cb915be11562 /drivers/scsi/scsi_transport_srp.c | |
parent | 73c655410181b7fc9a5ff321a5021a684ada99e7 (diff) |
brcmfmac: clear EAP/association status bits on linkdown events
This ensure that previous association attempts do not leave stale statuses
on subsequent attempts.
This fixes the WARN_ON(!cr->bss)) from __cfg80211_connect_result() when
connecting to an AP after a previous connection failure (e.g. where EAP fails
due to incorrect psk but association succeeded). In some scenarios, indeed,
brcmf_is_linkup() was reporting a link up event too early due to stale
BRCMF_VIF_STATUS_ASSOC_SUCCESS bit, thus reporting to cfg80211 a connection
result with a zeroed bssid (vif->profile.bssid is still empty), causing the
WARN_ON due to the call to cfg80211_get_bss() with the empty bssid.
Signed-off-by: Luca Pesce <luca.pesce@vimar.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1608807119-21785-1-git-send-email-luca.pesce@vimar.com
Diffstat (limited to 'drivers/scsi/scsi_transport_srp.c')
0 files changed, 0 insertions, 0 deletions