diff options
| author | Abhijeet Kolekar <abhijeet.kolekar@intel.com> | 2010-05-11 14:22:11 -0400 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2010-05-12 16:39:07 -0400 |
| commit | 058897a4e93a6fc6d331e2ef591b2d6571431265 (patch) | |
| tree | 4f6d88f66b9cfefe9f7931566773d885b247658d | |
| parent | ffdc4cbe5b17c83af779f45de8536c6ece297e42 (diff) | |
mac80211: fix paged defragmentation
Paged RX skb patch broke the defragmentation. We need to read hdr again
after linearization.
It fixes following bug
http://bugzilla.intellinuxwireless.org/show_bug.cgi?id=2194
Signed-off-by: Zhu, Yi <yi.zhu@intel.com>
Signed-off-by: Abhijeet Kolekar <abhijeet.kolekar@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
| -rw-r--r-- | net/mac80211/rx.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c index e4f325f68fd..8fa99554f4e 100644 --- a/net/mac80211/rx.c +++ b/net/mac80211/rx.c | |||
| @@ -1252,6 +1252,12 @@ ieee80211_rx_h_defragment(struct ieee80211_rx_data *rx) | |||
| 1252 | if (skb_linearize(rx->skb)) | 1252 | if (skb_linearize(rx->skb)) |
| 1253 | return RX_DROP_UNUSABLE; | 1253 | return RX_DROP_UNUSABLE; |
| 1254 | 1254 | ||
| 1255 | /* | ||
| 1256 | * skb_linearize() might change the skb->data and | ||
| 1257 | * previously cached variables (in this case, hdr) need to | ||
| 1258 | * be refreshed with the new data. | ||
| 1259 | */ | ||
| 1260 | hdr = (struct ieee80211_hdr *)rx->skb->data; | ||
| 1255 | seq = (sc & IEEE80211_SCTL_SEQ) >> 4; | 1261 | seq = (sc & IEEE80211_SCTL_SEQ) >> 4; |
| 1256 | 1262 | ||
| 1257 | if (frag == 0) { | 1263 | if (frag == 0) { |
