diff options
author | Johannes Berg <johannes.berg@intel.com> | 2014-01-22 10:05:27 +0100 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2014-02-04 21:48:16 +0100 |
commit | 1ff79dfa37d36c16d91eeeebb8cc3dcba32c2c16 (patch) | |
tree | 6400fb9a5f7470a37afc9fac5d8b9e07b2c23f39 /net/wireless | |
parent | c782bf8caae59a6cdd17ed1b99c126167dae49b2 (diff) | |
download | blackbird-op-linux-1ff79dfa37d36c16d91eeeebb8cc3dcba32c2c16.tar.gz blackbird-op-linux-1ff79dfa37d36c16d91eeeebb8cc3dcba32c2c16.zip |
nl80211: check channel switch validity better
Before allowing userspace to initiate a channel switch, check
that it's actually connected in some sense. Also use a more
appropriate error code for the not connected case.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/wireless')
-rw-r--r-- | net/wireless/nl80211.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c index a3515ebbd32b..d0f69f5523e8 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -5771,10 +5771,15 @@ static int nl80211_channel_switch(struct sk_buff *skb, struct genl_info *info) /* useless if AP is not running */ if (!wdev->beacon_interval) - return -EINVAL; + return -ENOTCONN; break; case NL80211_IFTYPE_ADHOC: + if (!wdev->ssid_len) + return -ENOTCONN; + break; case NL80211_IFTYPE_MESH_POINT: + if (!wdev->mesh_id_len) + return -ENOTCONN; break; default: return -EOPNOTSUPP; |