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 | |
| parent | mac80211: remove doubled semicolon (diff) | |
| download | linux-dev-57007121228347515187049a9c65f7cbeb923dd1.tar.xz linux-dev-57007121228347515187049a9c65f7cbeb923dd1.zip | |
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')
| -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 8e56eeb583aa..c5661c5ad8f3 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; } |