diff options
author | qingli <qingli@FreeBSD.org> | 2011-10-16 22:15:13 +0000 |
---|---|---|
committer | qingli <qingli@FreeBSD.org> | 2011-10-16 22:15:13 +0000 |
commit | dd2b5b91ebc744aa49cd0ed17b61b943d8b108e3 (patch) | |
tree | dc9c2a39d40c4fbf574ae9b4bdc3249b3aed5034 /sys/netinet6 | |
parent | 770f64229ce141801d6971ceea42082015323a53 (diff) | |
download | FreeBSD-src-dd2b5b91ebc744aa49cd0ed17b61b943d8b108e3.zip FreeBSD-src-dd2b5b91ebc744aa49cd0ed17b61b943d8b108e3.tar.gz |
The IPv6 code was influx at the time of r196865 due to the L2/L3
separation rewrite changes. r196865 was committed to fix a scope
violation problem in the following test scenario:
box-1# ifconfig em0 inet6 2001:db8:1:: prefixlen 64 anycast
box-1# ifconfig em1 inet6 2001:db8:2::1 prefixlen 64
box-2# ifconfig re0 inet6 2001:db8:1::6 prefixlen 64
em0 and re0 are on the same link.
box-2# ping6 2001:db8:1::
PING6(56=40+8+8 bytes) 2001:db8:1::6 --> 2001:db8:1::
the ICMPv6 response should have a source address of em1, which
is 2001:db8:2::1, not the link-local address of em0.
That code is no longer necessary and breaks the IPv6-Ready logo
testing, so revert it now.
Reviewed by: hrs
MFC after: 3 days
Diffstat (limited to 'sys/netinet6')
-rw-r--r-- | sys/netinet6/icmp6.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sys/netinet6/icmp6.c b/sys/netinet6/icmp6.c index 6a2da07..d22bc43 100644 --- a/sys/netinet6/icmp6.c +++ b/sys/netinet6/icmp6.c @@ -2244,10 +2244,6 @@ icmp6_reflect(struct mbuf *m, size_t off) } } - if ((srcp != NULL) && - (in6_addrscope(srcp) != in6_addrscope(&ip6->ip6_src))) - srcp = NULL; - if (srcp == NULL) { int e; struct sockaddr_in6 sin6; |