summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVeerendranath Jakkam <quic_vjakkam@quicinc.com>2024-09-04 16:20:21 +0200
committerJohannes Berg <johannes.berg@intel.com>2024-09-06 12:55:43 +0200
commitf54a1baee098170b13b624ca5ed2afdb4d29edbc (patch)
treeeb229853b14b24bd5af394c922a4697397aa0d49
parentwifi: cfg80211: fix UBSAN noise in cfg80211_wext_siwscan() (diff)
downloadlinux-f54a1baee098170b13b624ca5ed2afdb4d29edbc.tar.xz
linux-f54a1baee098170b13b624ca5ed2afdb4d29edbc.zip
wifi: cfg80211: Avoid RCU debug splat in __cfg80211_bss_update error paths
Replace rcu_dereference() with rcu_access_pointer() since we already hold the lock and own the 'tmp' at this point. This is needed to avoid suspicious rcu_dereference_check warnings in__cfg80211_bss_update error paths. Signed-off-by: Veerendranath Jakkam <quic_vjakkam@quicinc.com> Link: https://patch.msgid.link/20240904142021.3887360-1-quic_vjakkam@quicinc.com Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to '')
-rw-r--r--net/wireless/scan.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/net/wireless/scan.c b/net/wireless/scan.c
index 3ff818849d83..1ef6448bc636 100644
--- a/net/wireless/scan.c
+++ b/net/wireless/scan.c
@@ -2008,10 +2008,10 @@ __cfg80211_bss_update(struct cfg80211_registered_device *rdev,
return found;
free_ies:
- ies = (void *)rcu_dereference(tmp->pub.beacon_ies);
+ ies = (void *)rcu_access_pointer(tmp->pub.beacon_ies);
if (ies)
kfree_rcu(ies, rcu_head);
- ies = (void *)rcu_dereference(tmp->pub.proberesp_ies);
+ ies = (void *)rcu_access_pointer(tmp->pub.proberesp_ies);
if (ies)
kfree_rcu(ies, rcu_head);