diff options
| author | Srinivas Dasari <dasaris@codeaurora.org> | 2018-02-02 11:15:27 +0200 |
|---|---|---|
| committer | Min Liu <minliu@codeaurora.org> | 2019-01-10 11:16:59 +0800 |
| commit | 682aa32d0947f50f25587d2e2681a029fbe5c7ee (patch) | |
| tree | d85cdb69508ea5e16da06e1e568c2ea797e2c0ea | |
| parent | 3d2a81a6262967bc280d14be3b0ec23a4bbe8ca0 (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: Jouni Malinen <jouni@codeaurora.org>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Git-commit: db8d93a7a355121d49777c059afbca23c53c8628
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
CRs-Fixed: 2210017
Change-Id: I41f7c25d68a5060d7b5e24fb5bd164e063311ed0
Signed-off-by: Srinivas Dasari <dasaris@codeaurora.org>
Signed-off-by: Jiachao Wu <jiacwu@codeaurora.org>
Signed-off-by: Min Liu <minliu@codeaurora.org>
| -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 369829bf7188..b2d90f48227d 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -11173,7 +11173,7 @@ static int nl80211_external_auth(struct sk_buff *skb, struct genl_info *info) struct net_device *dev = info->user_ptr[1]; struct cfg80211_external_auth_params params; - if (rdev->ops->external_auth) + if (!rdev->ops->external_auth) return -EOPNOTSUPP; if (!info->attrs[NL80211_ATTR_SSID]) |