diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2007-03-10 10:40:27 -0500 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-04-26 01:24:37 -0400 |
commit | 39f69c6f922fbfb51e1ff24c9e196584a79f1484 (patch) | |
tree | 2d68a730ba5b89f895c28489872a1eea3b37db5a | |
parent | 48d49d0ccdaa9caff4636ef9c3410973d28131b5 (diff) |
[SK_BUFF] xfrm: Use skb_set_mac_header in the memmove cases
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv4/xfrm4_mode_tunnel.c | 6 | ||||
-rw-r--r-- | net/ipv6/xfrm6_mode_beet.c | 6 | ||||
-rw-r--r-- | net/ipv6/xfrm6_mode_tunnel.c | 6 |
3 files changed, 12 insertions, 6 deletions
diff --git a/net/ipv4/xfrm4_mode_tunnel.c b/net/ipv4/xfrm4_mode_tunnel.c index 4872b30ba5d8..f09055d3a768 100644 --- a/net/ipv4/xfrm4_mode_tunnel.c +++ b/net/ipv4/xfrm4_mode_tunnel.c | |||
@@ -91,6 +91,7 @@ static int xfrm4_tunnel_output(struct xfrm_state *x, struct sk_buff *skb) | |||
91 | static int xfrm4_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) | 91 | static int xfrm4_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) |
92 | { | 92 | { |
93 | struct iphdr *iph = skb->nh.iph; | 93 | struct iphdr *iph = skb->nh.iph; |
94 | const unsigned char *old_mac; | ||
94 | int err = -EINVAL; | 95 | int err = -EINVAL; |
95 | 96 | ||
96 | switch (iph->protocol){ | 97 | switch (iph->protocol){ |
@@ -125,8 +126,9 @@ static int xfrm4_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) | |||
125 | skb->protocol = htons(ETH_P_IPV6); | 126 | skb->protocol = htons(ETH_P_IPV6); |
126 | } | 127 | } |
127 | #endif | 128 | #endif |
128 | skb->mac.raw = memmove(skb->data - skb->mac_len, | 129 | old_mac = skb->mac.raw; |
129 | skb->mac.raw, skb->mac_len); | 130 | skb_set_mac_header(skb, -skb->mac_len); |
131 | memmove(skb->mac.raw, old_mac, skb->mac_len); | ||
130 | skb->nh.raw = skb->data; | 132 | skb->nh.raw = skb->data; |
131 | err = 0; | 133 | err = 0; |
132 | 134 | ||
diff --git a/net/ipv6/xfrm6_mode_beet.c b/net/ipv6/xfrm6_mode_beet.c index edcfffa9e87b..53cfe1a10ccd 100644 --- a/net/ipv6/xfrm6_mode_beet.c +++ b/net/ipv6/xfrm6_mode_beet.c | |||
@@ -59,6 +59,7 @@ static int xfrm6_beet_output(struct xfrm_state *x, struct sk_buff *skb) | |||
59 | static int xfrm6_beet_input(struct xfrm_state *x, struct sk_buff *skb) | 59 | static int xfrm6_beet_input(struct xfrm_state *x, struct sk_buff *skb) |
60 | { | 60 | { |
61 | struct ipv6hdr *ip6h; | 61 | struct ipv6hdr *ip6h; |
62 | const unsigned char *old_mac; | ||
62 | int size = sizeof(struct ipv6hdr); | 63 | int size = sizeof(struct ipv6hdr); |
63 | int err = -EINVAL; | 64 | int err = -EINVAL; |
64 | 65 | ||
@@ -69,8 +70,9 @@ static int xfrm6_beet_input(struct xfrm_state *x, struct sk_buff *skb) | |||
69 | memmove(skb->data, skb->nh.raw, size); | 70 | memmove(skb->data, skb->nh.raw, size); |
70 | skb->nh.raw = skb->data; | 71 | skb->nh.raw = skb->data; |
71 | 72 | ||
72 | skb->mac.raw = memmove(skb->data - skb->mac_len, | 73 | old_mac = skb->mac.raw; |
73 | skb->mac.raw, skb->mac_len); | 74 | skb_set_mac_header(skb, -skb->mac_len); |
75 | memmove(skb->mac.raw, old_mac, skb->mac_len); | ||
74 | 76 | ||
75 | ip6h = skb->nh.ipv6h; | 77 | ip6h = skb->nh.ipv6h; |
76 | ip6h->payload_len = htons(skb->len - size); | 78 | ip6h->payload_len = htons(skb->len - size); |
diff --git a/net/ipv6/xfrm6_mode_tunnel.c b/net/ipv6/xfrm6_mode_tunnel.c index 0bc866c0d83c..d2c560c181a1 100644 --- a/net/ipv6/xfrm6_mode_tunnel.c +++ b/net/ipv6/xfrm6_mode_tunnel.c | |||
@@ -86,6 +86,7 @@ static int xfrm6_tunnel_output(struct xfrm_state *x, struct sk_buff *skb) | |||
86 | static int xfrm6_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) | 86 | static int xfrm6_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) |
87 | { | 87 | { |
88 | int err = -EINVAL; | 88 | int err = -EINVAL; |
89 | const unsigned char *old_mac; | ||
89 | 90 | ||
90 | if (skb->nh.raw[IP6CB(skb)->nhoff] != IPPROTO_IPV6 | 91 | if (skb->nh.raw[IP6CB(skb)->nhoff] != IPPROTO_IPV6 |
91 | && skb->nh.raw[IP6CB(skb)->nhoff] != IPPROTO_IPIP) | 92 | && skb->nh.raw[IP6CB(skb)->nhoff] != IPPROTO_IPIP) |
@@ -107,8 +108,9 @@ static int xfrm6_tunnel_input(struct xfrm_state *x, struct sk_buff *skb) | |||
107 | ip6ip_ecn_decapsulate(skb); | 108 | ip6ip_ecn_decapsulate(skb); |
108 | skb->protocol = htons(ETH_P_IP); | 109 | skb->protocol = htons(ETH_P_IP); |
109 | } | 110 | } |
110 | skb->mac.raw = memmove(skb->data - skb->mac_len, | 111 | old_mac = skb->mac.raw; |
111 | skb->mac.raw, skb->mac_len); | 112 | skb_set_mac_header(skb, -skb->mac_len); |
113 | memmove(skb->mac.raw, old_mac, skb->mac_len); | ||
112 | skb->nh.raw = skb->data; | 114 | skb->nh.raw = skb->data; |
113 | err = 0; | 115 | err = 0; |
114 | 116 | ||