diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2018-01-16 23:20:22 +0100 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2019-05-02 17:01:08 +0200 |
| commit | 824ec6f7e674912e621a71159138ec93d272dc14 (patch) | |
| tree | cdadc99a3277d224b7aa5c8a938ca1cbe6a6b557 /net/wireless | |
| parent | bd2fd56f2a9e17cfccc9e2597cc8ec23892531d9 (diff) | |
cfg80211: fix station info handling bugs
commit 5762d7d3eda25c03cc2d9d45227be3f5ab6bec9e upstream.
Fix two places where the structure isn't initialized to zero,
and thus can't be filled properly by the driver.
Fixes: 4a4b8169501b ("cfg80211: Accept multiple RSSI thresholds for CQM")
Fixes: 9930380f0bd8 ("cfg80211: implement IWRATE")
Change-Id: I2cd090709e9349aa70a5196951e582a29679c4ab
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
[bwh: Backported to 3.16: drop change in cfg80211_cqm_rssi_update()]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'net/wireless')
| -rw-r--r-- | net/wireless/wext-compat.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/wireless/wext-compat.c b/net/wireless/wext-compat.c index 18a7a7b11..d7ab7f855 100644 --- a/net/wireless/wext-compat.c +++ b/net/wireless/wext-compat.c @@ -1287,8 +1287,7 @@ static int cfg80211_wext_giwrate(struct net_device *dev, { struct wireless_dev *wdev = dev->ieee80211_ptr; struct cfg80211_registered_device *rdev = wiphy_to_dev(wdev->wiphy); - /* we are under RTNL - globally locked - so can use a static struct */ - static struct station_info sinfo; + struct station_info sinfo = {}; u8 addr[ETH_ALEN]; int err; |