diff options
author | Srinivas Dasari <dasaris@codeaurora.org> | 2018-02-02 04:15:27 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2018-02-19 07:17:09 -0500 |
commit | db8d93a7a355121d49777c059afbca23c53c8628 (patch) | |
tree | 00840a3fec114eebb4a461692cf470db1aadff16 | |
parent | 7a9b3ec1e19f691f6e69429d851a4084b86e6219 (diff) |
nl80211: Fix external_auth check for offloaded authentication
Unfortunately removal of the ext_feature flag in the last revision of
the patch ended up negating the comparison and prevented the command
from being processed (either nl80211_external_auth() or
rdev_external_auth() returns -EOPNOTSUPP). Fix this by adding back the
lost '!'.
Fixes: 40cbfa90218b ("cfg80211/nl80211: Optional authentication offload to userspace")
Signed-off-by: Srinivas Dasari <dasaris@codeaurora.org>
Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-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 412ed8676306..c6f256b29c73 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c | |||
@@ -12484,7 +12484,7 @@ static int nl80211_external_auth(struct sk_buff *skb, struct genl_info *info) | |||
12484 | struct net_device *dev = info->user_ptr[1]; | 12484 | struct net_device *dev = info->user_ptr[1]; |
12485 | struct cfg80211_external_auth_params params; | 12485 | struct cfg80211_external_auth_params params; |
12486 | 12486 | ||
12487 | if (rdev->ops->external_auth) | 12487 | if (!rdev->ops->external_auth) |
12488 | return -EOPNOTSUPP; | 12488 | return -EOPNOTSUPP; |
12489 | 12489 | ||
12490 | if (!info->attrs[NL80211_ATTR_SSID]) | 12490 | if (!info->attrs[NL80211_ATTR_SSID]) |