diff options
| author | Hiroaki SHIMODA <shimoda.hiroaki@gmail.com> | 2011-03-09 21:09:58 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-03-14 02:37:49 +0100 |
| commit | 46af31800b6916c92fffa529dc3c357008da957d (patch) | |
| tree | 2c66ee7b78527da504a294f1bd8a2a899623fead /net/ipv4/inetpeer.c | |
| parent | decnet: Convert to use flowidn where applicable. (diff) | |
| download | linux-46af31800b6916c92fffa529dc3c357008da957d.tar.xz linux-46af31800b6916c92fffa529dc3c357008da957d.zip | |
ipv4: Fix PMTU update.
On current net-next-2.6, when Linux receives ICMP Type: 3, Code: 4
(Destination unreachable (Fragmentation needed)),
icmp_unreach
-> ip_rt_frag_needed
(peer->pmtu_expires is set here)
-> tcp_v4_err
-> do_pmtu_discovery
-> ip_rt_update_pmtu
(peer->pmtu_expires is already set,
so check_peer_pmtu is skipped.)
-> check_peer_pmtu
check_peer_pmtu is skipped and MTU is not updated.
To fix this, let check_peer_pmtu execute unconditionally.
And some minor fixes
1) Avoid potential peer->pmtu_expires set to be zero.
2) In check_peer_pmtu, argument of time_before is reversed.
3) check_peer_pmtu expects peer->pmtu_orig is initialized as zero,
but not initialized.
Signed-off-by: Hiroaki SHIMODA <shimoda.hiroaki@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/inetpeer.c')
| -rw-r--r-- | net/ipv4/inetpeer.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv4/inetpeer.c b/net/ipv4/inetpeer.c index 6442c35edb0b..86b1d0878dff 100644 --- a/net/ipv4/inetpeer.c +++ b/net/ipv4/inetpeer.c @@ -511,6 +511,7 @@ struct inet_peer *inet_getpeer(struct inetpeer_addr *daddr, int create) p->rate_tokens = 0; p->rate_last = 0; p->pmtu_expires = 0; + p->pmtu_orig = 0; memset(&p->redirect_learned, 0, sizeof(p->redirect_learned)); INIT_LIST_HEAD(&p->unused); |