diff options
author | Patrick McHardy <kaber@trash.net> | 2008-07-08 10:21:27 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-07-08 10:21:27 (GMT) |
commit | acc81e1465d29e0284008770cc4b8bc90bd93bd7 (patch) | |
tree | 7580b619f82a6ae13c425051c305b7d5501e201d /include/linux/if_vlan.h | |
parent | 256f3bfce87fe39ec008b2c27890b30066984f50 (diff) | |
download | linux-fsl-qoriq-acc81e1465d29e0284008770cc4b8bc90bd93bd7.tar.xz |
vlan: fix network_header/mac_header adjustments
Lennert Buytenhek points out that the VLAN code incorrectly adjusts
skb->network_header to point in the middle of the VLAN header and
additionally tries to adjust skb->mac_header without checking for
validity.
The network_header should not be touched at all since we're only
adding headers in front of it, mac_header adjustments are not
necessary at all.
Based on patch by Lennert Buytenhek <buytenh@wantstofly.org>.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/if_vlan.h')
-rw-r--r-- | include/linux/if_vlan.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/linux/if_vlan.h b/include/linux/if_vlan.h index 5190452..8f5bf9b 100644 --- a/include/linux/if_vlan.h +++ b/include/linux/if_vlan.h @@ -279,8 +279,6 @@ static inline struct sk_buff *__vlan_put_tag(struct sk_buff *skb, unsigned short veth->h_vlan_TCI = htons(tag); skb->protocol = htons(ETH_P_8021Q); - skb->mac_header -= VLAN_HLEN; - skb->network_header -= VLAN_HLEN; return skb; } |