diff options
| author |   Simon Horman <horms@verge.net.au> | 2013-05-28 20:34:26 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2013-05-28 23:49:07 -0700 |
| commit | 29a3cad5c6ae9e7fbf1509d01d39c3c3c38f11f9 (patch) | |
| tree | fd6436d8a820801843ca257807c8af0f36425291 /net/ipv6/output_core.c | |
| parent | net: Correct comparisons and calculations using skb->tail and skb-transport_header (diff) | |
| download | linux-dev-29a3cad5c6ae9e7fbf1509d01d39c3c3c38f11f9.tar.xz linux-dev-29a3cad5c6ae9e7fbf1509d01d39c3c3c38f11f9.zip | |
ipv6: Correct comparisons and calculations using skb->tail and skb-transport_header
This corrects an regression introduced by "net: Use 16bits for *_headers
fields of struct skbuff" when NET_SKBUFF_DATA_USES_OFFSET is not set. In
that case skb->tail will be a pointer whereas skb->transport_header
will be an offset from head. This is corrected by using wrappers that
ensure that comparisons and calculations are always made using pointers.
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/output_core.c')
| -rw-r--r-- | net/ipv6/output_core.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ipv6/output_core.c b/net/ipv6/output_core.c index c2e73e647e44..ab92a3673fbb 100644 --- a/net/ipv6/output_core.c +++ b/net/ipv6/output_core.c @@ -40,7 +40,8 @@ int ip6_find_1stfragopt(struct sk_buff *skb, u8 **nexthdr) u16 offset = sizeof(struct ipv6hdr); struct ipv6_opt_hdr *exthdr = (struct ipv6_opt_hdr *)(ipv6_hdr(skb) + 1); - unsigned int packet_len = skb->tail - skb->network_header; + unsigned int packet_len = skb_tail_pointer(skb) - + skb_network_header(skb); int found_rhdr = 0; *nexthdr = &ipv6_hdr(skb)->nexthdr; |