diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2007-03-10 16:40:59 -0500 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-04-26 01:24:48 -0400 |
commit | 7e28ecc282574a7d72ace365fc9bc86e27ba880f (patch) | |
tree | 6321d16c591694d6405b723f7bea80855b8040ce /net/ipx/af_ipx.c | |
parent | e2d1bca7e6134671bcb19810d004a252aa6a644d (diff) |
[SK_BUFF]: Use skb_reset_network_header where the skb_pull return was being used
But only in the cases where its a newly allocated skb, i.e. one where skb->tail
is equal to skb->data, or just after skb_reserve, where this requirement is
maintained.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipx/af_ipx.c')
-rw-r--r-- | net/ipx/af_ipx.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ipx/af_ipx.c b/net/ipx/af_ipx.c index 6c6c0a3a0ab5..9aa7b961b3eb 100644 --- a/net/ipx/af_ipx.c +++ b/net/ipx/af_ipx.c | |||
@@ -576,7 +576,9 @@ static struct sk_buff *ipxitf_adjust_skbuff(struct ipx_interface *intrfc, | |||
576 | skb2 = alloc_skb(len, GFP_ATOMIC); | 576 | skb2 = alloc_skb(len, GFP_ATOMIC); |
577 | if (skb2) { | 577 | if (skb2) { |
578 | skb_reserve(skb2, out_offset); | 578 | skb_reserve(skb2, out_offset); |
579 | skb2->nh.raw = skb2->h.raw = skb_put(skb2, skb->len); | 579 | skb_reset_network_header(skb2); |
580 | skb2->h.raw = skb2->data; | ||
581 | skb_put(skb2, skb->len); | ||
580 | memcpy(ipx_hdr(skb2), ipx_hdr(skb), skb->len); | 582 | memcpy(ipx_hdr(skb2), ipx_hdr(skb), skb->len); |
581 | memcpy(skb2->cb, skb->cb, sizeof(skb->cb)); | 583 | memcpy(skb2->cb, skb->cb, sizeof(skb->cb)); |
582 | } | 584 | } |