linux-stable.git/net/wireless/util.c, branch linux-4.6.y

cfg80211: handle failed skb allocation

2016-08-10T10:54:49+00:00

commit 16a910a6722b7a8680409e634c7c0dac073c01e4 upstream.

Handle the case when dev_alloc_skb returns NULL.

Fixes: 2b67f944f88c2 ("cfg80211: reuse existing page fragments in A-MSDU rx")
Signed-off-by: Gregory Greenman 
Signed-off-by: Luca Coelho 
Signed-off-by: Johannes Berg 
Signed-off-by: Greg Kroah-Hartman

cfg80211: fix proto in ieee80211_data_to_8023 for frames without LLC header

2016-07-27T15:42:07+00:00

commit c041778c966c92c964033f1cdfee60a9f2b5e465 upstream.

The PDU length of incoming LLC frames is set to the total skb payload size
in __ieee80211_data_to_8023() of net/wireless/util.c which incorrectly
includes the length of the IEEE 802.11 header.

The resulting LLC frame header has a too large PDU length, causing the
llc_fixup_skb() function of net/llc/llc_input.c to reject the incoming
skb, effectively breaking STP.

Solve the problem by properly substracting the IEEE 802.11 frame header size
from the PDU length, allowing the LLC processor to pick up the incoming
control messages.

Special thanks to Gerry Rozema for tracking down the regression and proposing
a suitable patch.

Fixes: 2d1c304cb2d5 ("cfg80211: add function for 802.3 conversion with separate output buffer")
Reported-by: Gerry Rozema 
Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg 
Signed-off-by: Greg Kroah-Hartman

wireless: use reset to set mac header

2016-03-05T03:45:13+00:00

Since offset is zero, it's not necessary to use set function. Reset
function is straightforward, and will remove the unnecessary add
operation in set function.

Signed-off-by: Zhang Shengju 
Signed-off-by: David S. Miller

cfg80211: reuse existing page fragments in A-MSDU rx

2016-02-24T08:04:37+00:00

This massively reduces data copying and thus improves rx performance

Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg

cfg80211: fix faulty variable initialization in ieee80211_amsdu_to_8023s

2016-02-24T08:04:37+00:00

reuse_skb is set to true if the code decides to use the last segment.
Fixes a memory leak

Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg

cfg80211: add support for non-linear skbs in ieee80211_amsdu_to_8023s

2016-02-24T08:04:34+00:00

Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg

cfg80211: add function for 802.3 conversion with separate output buffer

2016-02-24T08:04:33+00:00

Use skb_copy_bits in preparation for allowing fragmented skbs

Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg

mac80211: move A-MSDU skb_linearize call to ieee80211_amsdu_to_8023s

2016-02-24T08:04:33+00:00

Prepararation for zero-copy A-MSDU support with page fragment SKBs

Signed-off-by: Felix Fietkau 
Signed-off-by: Johannes Berg

cfg80211: Remove unused cfg80211_can_use_iftype_chan()

2015-12-04T13:43:32+00:00

Last caller of this function was removed in 3.17 in commit
97dc94f1d933c9df2c0b327066ea130c0e92083f.

Signed-off-by: Michal Sojka 
Signed-off-by: Johannes Berg

cfg80211: replace ieee80211_ie_split() with an inline

2015-12-04T13:43:32+00:00

The function is a very simple wrapper around another one,
just adds a few default parameters, so replace it with a
static inline instead of using EXPORT_SYMBOL, reducing
the module size slightly.

Signed-off-by: Johannes Berg