summaryrefslogtreecommitdiffstats
path: root/net/wireless
diff options
context:
space:
mode:
authorBen Greear <greearb@candelatech.com>2013-06-19 14:06:26 -0700
committerJohannes Berg <johannes.berg@intel.com>2013-06-24 15:55:36 +0200
commitf9bef3df52fe61067e4c1c6cfb2037cb6b259a6a (patch)
tree80d8f1dfd41630b161095f301d4f158838dc73c0 /net/wireless
parent0e3a39b5620bc84f25ffb0592b05b0350e8b0520 (diff)
downloadop-kernel-dev-f9bef3df52fe61067e4c1c6cfb2037cb6b259a6a.zip
op-kernel-dev-f9bef3df52fe61067e4c1c6cfb2037cb6b259a6a.tar.gz
wireless: check for dangling wdev->current_bss pointer
If it *is* still set when the netdev is being deleted, then we are about to leak a pointer. Warn and clean up in that case. Signed-off-by: Ben Greear <greearb@candelatech.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/wireless')
-rw-r--r--net/wireless/core.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/net/wireless/core.c b/net/wireless/core.c
index 4224e75..672459b 100644
--- a/net/wireless/core.c
+++ b/net/wireless/core.c
@@ -934,6 +934,12 @@ static int cfg80211_netdev_notifier_call(struct notifier_block *nb,
* freed.
*/
cfg80211_process_wdev_events(wdev);
+
+ if (WARN_ON(wdev->current_bss)) {
+ cfg80211_unhold_bss(wdev->current_bss);
+ cfg80211_put_bss(wdev->wiphy, &wdev->current_bss->pub);
+ wdev->current_bss = NULL;
+ }
break;
case NETDEV_PRE_UP:
if (!(wdev->wiphy->interface_modes & BIT(wdev->iftype)))
OpenPOWER on IntegriCloud