diff options
author | Johannes Berg <johannes.berg@intel.com> | 2015-01-16 21:05:02 +0100 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2015-01-16 21:05:52 +0100 |
commit | 57007121228347515187049a9c65f7cbeb923dd1 (patch) | |
tree | 6182ff537e61976ac4669c09b8bba37a8cfdd248 /net/wireless | |
parent | 5e06a9e8b6db441371dadc02f3e2348196f408dd (diff) | |
download | op-kernel-dev-57007121228347515187049a9c65f7cbeb923dd1.zip op-kernel-dev-57007121228347515187049a9c65f7cbeb923dd1.tar.gz |
cfg80211: fix checking nl80211_send_station() return value
The return value from nl80211_send_station() is the length of the
skb, or a negative error, so abort sending the message only when
the return value was negative.
This fixes the ibss_rsn wpa_supplicant test case.
Reported-by: Jouni Malinen <j@w1.fi>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/wireless')
-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 8e56eeb..c5661c5 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -11788,7 +11788,7 @@ void cfg80211_del_sta_sinfo(struct net_device *dev, const u8 *mac_addr, return; if (nl80211_send_station(msg, NL80211_CMD_DEL_STATION, 0, 0, 0, - rdev, dev, mac_addr, sinfo)) { + rdev, dev, mac_addr, sinfo) < 0) { nlmsg_free(msg); return; } |