Message ID | 4DAFF122.7010608@secunet.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: Thomas Egerer <thomas.egerer@secunet.com> Date: Thu, 21 Apr 2011 10:56:02 +0200 > > The changes introduced with git-commit a02e4b7d missed to remove the > hoplimit initialization. As a result, ipv6_get_mtu interprets the return > value of dst_metric_raw (-1) as 255 and answers ping6 with this hoplimit. > This patche removes the line such that ping6 is answered with the > hoplimit value configured via sysctl. > > Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com> Sorry for breaking this, patch applied. Thanks! -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/net/ipv6/route.c b/net/ipv6/route.c index 843406f..0a5d02a 100644 --- a/net/ipv6/route.c +++ b/net/ipv6/route.c @@ -2012,7 +2012,6 @@ struct rt6_info *addrconf_dst_alloc(struct inet6_dev *idev, rt->dst.output = ip6_output; rt->rt6i_dev = net->loopback_dev; rt->rt6i_idev = idev; - dst_metric_set(&rt->dst, RTAX_HOPLIMIT, -1); rt->dst.obsolete = -1; rt->rt6i_flags = RTF_UP | RTF_NONEXTHOP;
The changes introduced with git-commit a02e4b7d missed to remove the hoplimit initialization. As a result, ipv6_get_mtu interprets the return value of dst_metric_raw (-1) as 255 and answers ping6 with this hoplimit. This patche removes the line such that ping6 is answered with the hoplimit value configured via sysctl. Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com> --- net/ipv6/route.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-)