nl80211: fix per-station group key get/del and memory leak
authorJohannes Berg <johannes.berg@intel.com>
Fri, 23 Jan 2015 10:10:12 +0000 (11:10 +0100)
committerDanny Wood <danwood76@gmail.com>
Tue, 29 Jan 2019 13:06:19 +0000 (13:06 +0000)
commit297e98766572eba6b9f4efffcd19ac0f7c5a8bc7
treec4dcb77872a34baf4011c67ee4b81bc1dd477b6e
parent4fa61e1ada17621e33321b052d65f2f3bcf84349
nl80211: fix per-station group key get/del and memory leak

commit 0fa7b39131576dd1baa6ca17fca53c65d7f62249 upstream.

In case userspace attempts to obtain key information for or delete a
unicast key, this is currently erroneously rejected unless the driver
sets the WIPHY_FLAG_IBSS_RSN flag. Apparently enough drivers do so it
was never noticed.

Fix that, and while at it fix a potential memory leak: the error path
in the get_key() function was placed after allocating a message but
didn't free it - move it to a better place. Luckily admin permissions
are needed to call this operation.

Fixes: e31b82136d1ad ("cfg80211/mac80211: allow per-station GTKs")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
net/wireless/nl80211.c