diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2016-09-13 15:53:55 +0200 |
|---|---|---|
| committer | Sasha Levin <alexander.levin@verizon.com> | 2016-10-02 21:08:49 -0400 |
| commit | adb108bb9a38c435a667c6c14cf5da3da58025ba (patch) | |
| tree | ee3957489b332eb7b7c4574ad274c92235369279 /net | |
| parent | 14279ced5911c18ca26b70309c3dbff06d248592 (diff) | |
nl80211: validate number of probe response CSA counters
[ Upstream commit ad5987b47e96a0fb6d13fea250e936aed000093c ]
Due to an apparent copy/paste bug, the number of counters for the
beacon configuration were checked twice, instead of checking the
number of probe response counters. Fix this to check the number of
probe response counters before parsing those.
Cc: stable@vger.kernel.org
Fixes: 9a774c78e211 ("cfg80211: Support multiple CSA counters")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/wireless/nl80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c index f24138681b80..978d7f91ca91 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -6466,7 +6466,7 @@ static int nl80211_channel_switch(struct sk_buff *skb, struct genl_info *info) params.n_counter_offsets_presp = len / sizeof(u16); if (rdev->wiphy.max_num_csa_counters && - (params.n_counter_offsets_beacon > + (params.n_counter_offsets_presp > rdev->wiphy.max_num_csa_counters)) return -EINVAL; |