diff options
| author | Veerendranath Jakkam <quic_vjakkam@quicinc.com> | 2022-08-05 16:52:59 +0300 |
|---|---|---|
| committer | Kalle Valo <kvalo@kernel.org> | 2022-08-08 11:09:52 +0300 |
| commit | baa56dfe2cdad12edb2625b2d454e205943c3402 (patch) | |
| tree | ea0ee6a50db97a1e7e5e4ad969587295527ce184 /drivers/net/wireless/microchip | |
| parent | ac0dbed9ba4c38ed9b5fd3a43ee4bc1f48901a34 (diff) | |
| download | linux-baa56dfe2cdad12edb2625b2d454e205943c3402.tar.xz | |
wifi: cfg80211: Fix validating BSS pointers in __cfg80211_connect_result
Driver's SME is allowed to fill either BSSID or BSS pointers in struct
cfg80211_connect_resp_params when indicating connect response but a
check in __cfg80211_connect_result() is giving unnecessary warning when
driver's SME fills only BSSID pointer and not BSS pointer in struct
cfg80211_connect_resp_params.
In case of mac80211 with auth/assoc path, it is always expected to fill
BSS pointers in struct cfg80211_connect_resp_params when calling
__cfg80211_connect_result() since cfg80211 must have hold BSS pointers
in cfg80211_mlme_assoc().
So, skip the check for the drivers which support cfg80211 connect
callback, for example with brcmfmac is one such driver which had the
warning:
WARNING: CPU: 5 PID: 514 at net/wireless/sme.c:786 __cfg80211_connect_result+0x2fc/0x5c0 [cfg80211]
Reported-by: Linus Torvalds <torvalds@linux-foundation.org>
Fixes: efbabc116500 ("cfg80211: Indicate MLO connection info in connect and roam callbacks")
Signed-off-by: Veerendranath Jakkam <quic_vjakkam@quicinc.com>
[kvalo@kernel.org: add more info to the commit log]
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220805135259.4126630-1-quic_vjakkam@quicinc.com
Diffstat (limited to 'drivers/net/wireless/microchip')
0 files changed, 0 insertions, 0 deletions