mac80211: fix tx-mgmt cookie value being left uninitialized
authorNicolas Cavallari <Nicolas.Cavallari@lri.fr>
Mon, 16 Jul 2012 16:36:52 +0000 (18:36 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 17 Jul 2012 09:22:42 +0000 (11:22 +0200)
commit "mac80211: unify SW/offload remain-on-channel"
moved the cookie assignment from ieee80211_mgmt_tx()
to ieee80211_start_roc_work().  But the latter is only
called where offchannel is needed.  If offchannel isn't
needed/used, a uninitialized cookie value would be returned
to userspace.

This patch sets the cookie value when offchannel isn't used.

Signed-off-by: Nicolas Cavallari <cavallar@lri.fr>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/cfg.c

index 10dd9631e4da4aed838f4e177234a4902d8cbf91..efbbdc8a2be08154380851f53de968da3b88c5a5 100644 (file)
@@ -2493,6 +2493,7 @@ static int ieee80211_mgmt_tx(struct wiphy *wiphy, struct wireless_dev *wdev,
        skb->dev = sdata->dev;
 
        if (!need_offchan) {
+               *cookie = (unsigned long) skb;
                ieee80211_tx_skb(sdata, skb);
                ret = 0;
                goto out_unlock;