diff options
| author | Nicolas Cavallari <Nicolas.Cavallari@lri.fr> | 2012-07-16 18:36:52 +0200 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2012-07-17 11:22:42 +0200 |
| commit | 7f9f78ab96ebdb3533acd791efe485b25995947e (patch) | |
| tree | 82d4c54d095ba8107e3841036e50e325254dd39d | |
| parent | 4290cb4bf212112e3d6f860e25f000ca8a1ca6a4 (diff) | |
| download | op-kernel-dev-7f9f78ab96ebdb3533acd791efe485b25995947e.zip op-kernel-dev-7f9f78ab96ebdb3533acd791efe485b25995947e.tar.gz | |
mac80211: fix tx-mgmt cookie value being left uninitialized
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>
| -rw-r--r-- | net/mac80211/cfg.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index 10dd963..efbbdc8 100644 --- a/net/mac80211/cfg.c +++ b/net/mac80211/cfg.c @@ -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; |
