Skip to content

Commit

Permalink
UPSTREAM: cfg80211: don't WARN about two consecutive Country IE hint
Browse files Browse the repository at this point in the history
This can happen and there is no point in added more
detection code lower in the stack. Catching these in one
single point (cfg80211) is enough. Stop WARNING about this
case.

This fixes:
https://bugzilla.kernel.org/show_bug.cgi?id=89001

Cc: stable@vger.kernel.org
Fixes: 2f1c6c5 ("cfg80211: process non country IE conflicting first")
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
(cherry picked from commit 70dcec5)

BUG=chrome-os-partner:47771
TEST=manual

Change-Id: I9055277f54bd1742b808d434bb5f14de55d408d4
Signed-off-by: Kevin Cernekee <cernekee@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/314512
  • Loading branch information
Emmanuel Grumbach authored and ChromeOS bot committed Nov 26, 2015
1 parent 8940057 commit 5283e59
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions net/wireless/reg.c
Original file line number Diff line number Diff line change
Expand Up @@ -1845,11 +1845,8 @@ __reg_process_hint_country_ie(struct wiphy *wiphy,
return REG_REQ_IGNORE;
return REG_REQ_ALREADY_SET;
}
/*
* Two consecutive Country IE hints on the same wiphy.
* This should be picked up early by the driver/stack
*/
if (WARN_ON(regdom_changes(country_ie_request->alpha2)))

if (regdom_changes(country_ie_request->alpha2))
return REG_REQ_OK;
return REG_REQ_ALREADY_SET;
}
Expand Down

0 comments on commit 5283e59

Please sign in to comment.