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)
committerJohannes Berg <johannes.berg@intel.com>
Fri, 23 Jan 2015 10:21:02 +0000 (11:21 +0100)
commit0fa7b39131576dd1baa6ca17fca53c65d7f62249
tree75279e79f8ffd5dd26ba3dd1e497dae83d5705d8
parent3a5c5e81d8128a9e43abc52b75dd21d3da7a0cfc
nl80211: fix per-station group key get/del and memory leak

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.

Cc: stable@vger.kernel.org
Fixes: e31b82136d1ad ("cfg80211/mac80211: allow per-station GTKs")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/nl80211.c