diff options
| author | Nicolas Dichtel <nicolas.dichtel@6wind.com> | 2015-11-27 18:17:05 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-12-01 15:07:59 -0500 |
| commit | 304d888b29cf96f1dd53511ee686499cd8cdf249 (patch) | |
| tree | b7c215879c5b0b71bb959d13d13b0e1db0fd0caf /net/ipv6/addrconf.c | |
| parent | 142a2e7ece8d8ac0e818eb2c91f99ca894730e2a (diff) | |
| download | lwn-304d888b29cf96f1dd53511ee686499cd8cdf249.tar.gz lwn-304d888b29cf96f1dd53511ee686499cd8cdf249.zip | |
Revert "ipv6: ndisc: inherit metadata dst when creating ndisc requests"
This reverts commit ab450605b35caa768ca33e86db9403229bf42be4.
In IPv6, we cannot inherit the dst of the original dst. ndisc packets
are IPv6 packets and may take another route than the original packet.
This patch breaks the following scenario: a packet comes from eth0 and
is forwarded through vxlan1. The encapsulated packet triggers an NS
which cannot be sent because of the wrong route.
CC: Jiri Benc <jbenc@redhat.com>
CC: Thomas Graf <tgraf@suug.ch>
Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/addrconf.c')
| -rw-r--r-- | net/ipv6/addrconf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index d84742f003a9..61f26851655c 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c @@ -3642,7 +3642,7 @@ static void addrconf_dad_work(struct work_struct *w) /* send a neighbour solicitation for our addr */ addrconf_addr_solict_mult(&ifp->addr, &mcaddr); - ndisc_send_ns(ifp->idev->dev, &ifp->addr, &mcaddr, &in6addr_any, NULL); + ndisc_send_ns(ifp->idev->dev, &ifp->addr, &mcaddr, &in6addr_any); out: in6_ifa_put(ifp); rtnl_unlock(); |