提交 e367c2d0 编写于 作者: L lucien 提交者: David S. Miller

ipv6: ip6_append_data_mtu do not handle the mtu of the second fragment properly

In ip6_append_data_mtu(), when the xfrm mode is not tunnel(such as
transport),the ipsec header need to be added in the first fragment, so the mtu
will decrease to reserve space for it, then the second fragment come, the mtu
should be turn back, as the commit 0c183379
said.  however, in the commit a493e60ac4bbe2e977e7129d6d8cbb0dd236be, it use
*mtu = min(*mtu, ...) to change the mtu, which lead to the new mtu is alway
equal with the first fragment's. and cannot turn back.

when I test through  ping6 -c1 -s5000 $ip (mtu=1280):
...frag (0|1232) ESP(spi=0x00002000,seq=0xb), length 1232
...frag (1232|1216)
...frag (2448|1216)
...frag (3664|1216)
...frag (4880|164)

which should be:
...frag (0|1232) ESP(spi=0x00001000,seq=0x1), length 1232
...frag (1232|1232)
...frag (2464|1232)
...frag (3696|1232)
...frag (4928|116)

so delete the min() when change back the mtu.
Signed-off-by: NXin Long <lucien.xin@gmail.com>
Fixes: 75a493e6 ("ipv6: ip6_append_data_mtu did not care about pmtudisc and frag_size")
Acked-by: NHannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: NDavid S. Miller <davem@davemloft.net>
上级 d1958f8c
...@@ -1101,21 +1101,19 @@ static void ip6_append_data_mtu(unsigned int *mtu, ...@@ -1101,21 +1101,19 @@ static void ip6_append_data_mtu(unsigned int *mtu,
unsigned int fragheaderlen, unsigned int fragheaderlen,
struct sk_buff *skb, struct sk_buff *skb,
struct rt6_info *rt, struct rt6_info *rt,
bool pmtuprobe) unsigned int orig_mtu)
{ {
if (!(rt->dst.flags & DST_XFRM_TUNNEL)) { if (!(rt->dst.flags & DST_XFRM_TUNNEL)) {
if (skb == NULL) { if (skb == NULL) {
/* first fragment, reserve header_len */ /* first fragment, reserve header_len */
*mtu = *mtu - rt->dst.header_len; *mtu = orig_mtu - rt->dst.header_len;
} else { } else {
/* /*
* this fragment is not first, the headers * this fragment is not first, the headers
* space is regarded as data space. * space is regarded as data space.
*/ */
*mtu = min(*mtu, pmtuprobe ? *mtu = orig_mtu;
rt->dst.dev->mtu :
dst_mtu(rt->dst.path));
} }
*maxfraglen = ((*mtu - fragheaderlen) & ~7) *maxfraglen = ((*mtu - fragheaderlen) & ~7)
+ fragheaderlen - sizeof(struct frag_hdr); + fragheaderlen - sizeof(struct frag_hdr);
...@@ -1132,7 +1130,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to, ...@@ -1132,7 +1130,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to,
struct ipv6_pinfo *np = inet6_sk(sk); struct ipv6_pinfo *np = inet6_sk(sk);
struct inet_cork *cork; struct inet_cork *cork;
struct sk_buff *skb, *skb_prev = NULL; struct sk_buff *skb, *skb_prev = NULL;
unsigned int maxfraglen, fragheaderlen, mtu; unsigned int maxfraglen, fragheaderlen, mtu, orig_mtu;
int exthdrlen; int exthdrlen;
int dst_exthdrlen; int dst_exthdrlen;
int hh_len; int hh_len;
...@@ -1214,6 +1212,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to, ...@@ -1214,6 +1212,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to,
dst_exthdrlen = 0; dst_exthdrlen = 0;
mtu = cork->fragsize; mtu = cork->fragsize;
} }
orig_mtu = mtu;
hh_len = LL_RESERVED_SPACE(rt->dst.dev); hh_len = LL_RESERVED_SPACE(rt->dst.dev);
...@@ -1311,8 +1310,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to, ...@@ -1311,8 +1310,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to,
if (skb == NULL || skb_prev == NULL) if (skb == NULL || skb_prev == NULL)
ip6_append_data_mtu(&mtu, &maxfraglen, ip6_append_data_mtu(&mtu, &maxfraglen,
fragheaderlen, skb, rt, fragheaderlen, skb, rt,
np->pmtudisc >= orig_mtu);
IPV6_PMTUDISC_PROBE);
skb_prev = skb; skb_prev = skb;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册