Patch "Revert "ipv6: ndisc: inherit metadata dst when creating ndisc requests"" has been added to the 4.3-stable tree
Subsystems (3)
This is a note to let you know that I've just added the patch titled
Revert "ipv6: ndisc: inherit metadata dst when creating ndisc requests"
to the 4.3-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
revert-ipv6-ndisc-inherit-metadata-dst-when-creating-ndisc-requests.patch
and it can be found in the queue-4.3 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From foo@baz Fri Dec 11 11:38:06 EST 2015
From: Nicolas Dichtel <[email protected]>
Date: Fri, 27 Nov 2015 18:17:05 +0100
Subject: Revert "ipv6: ndisc: inherit metadata dst when creating ndisc requests"
From: Nicolas Dichtel <[email protected]>
[ Upstream commit 304d888b29cf96f1dd53511ee686499cd8cdf249 ]
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.
--- include/net/ndisc.h | 3 +-- net/ipv6/addrconf.c | 2 +- net/ipv6/ndisc.c | 10 +++------- net/ipv6/route.c | 2 +- 4 files changed, 6 insertions(+), 11 deletions(-)
include/net/ndisc.h +1 -2
@@ -182,8 +182,7 @@ int ndisc_rcv(struct sk_buff *skb);void ndisc_send_ns (struct net_device *dev,struct neighbour *neigh,const struct in6_addr *solicit, -const struct in6_addr *daddr,const struct in6_addr *saddr, -struct sk_buff *oskb); +const struct in6_addr *daddr,const struct in6_addr *saddr);void ndisc_send_rs (struct net_device *dev,const struct in6_addr *saddr,const struct in6_addr *daddr);
net/ipv6/addrconf.c +1 -1
@@ -3628,7 +3628,7 @@ static void addrconf_dad_work(struct wor/* send a neighbour solicitation for our addr */ addrconf_addr_solict_mult (&ifp->addr, &mcaddr); -ndisc_send_ns (ifp->idev->dev,NULL , &ifp->addr, &mcaddr, &in6addr_any,NULL ); +ndisc_send_ns (ifp->idev->dev,NULL , &ifp->addr, &mcaddr, &in6addr_any); out:in6_ifa_put (ifp);rtnl_unlock ();
net/ipv6/ndisc.c +3 -7
@@ -553,8 +553,7 @@ static void ndisc_send_unsol_na(struct nvoid ndisc_send_ns (struct net_device *dev,struct neighbour *neigh,const struct in6_addr *solicit, -const struct in6_addr *daddr,const struct in6_addr *saddr, -struct sk_buff *oskb) +const struct in6_addr *daddr,const struct in6_addr *saddr) {struct sk_buff *skb;struct in6_addr addr_buf;
@@ -590,9 +589,6 @@ void ndisc_send_ns(struct net_device *dendisc_fill_addr_option (skb,ND_OPT_SOURCE_LL_ADDR , dev->dev_addr); -if (!(dev->priv_flags &IFF_XMIT_DST_RELEASE ) && oskb) -skb_dst_copy (skb, oskb); -ndisc_send_skb (skb, daddr, saddr); }
@@ -679,12 +675,12 @@ static void ndisc_solicit(struct neighbo"%s: trying to ucast probe in NUD_INVALID: %pI6\n" , __func__, target); } -ndisc_send_ns (dev, neigh, target, target, saddr, skb); +ndisc_send_ns (dev, neigh, target, target, saddr); }else if ((probes -=NEIGH_VAR (neigh->parms,APP_PROBES )) <0 ) {neigh_app_ns (neigh); }else {addrconf_addr_solict_mult (target, &mcaddr); -ndisc_send_ns (dev,NULL , target, &mcaddr, saddr, skb); +ndisc_send_ns (dev,NULL , target, &mcaddr, saddr); } }
net/ipv6/route.c +1 -1
@@ -546,7 +546,7 @@ static void rt6_probe_deferred(struct wo container_of(w, struct __rt6_probe_work, work);addrconf_addr_solict_mult (&work->target, &mcaddr); -ndisc_send_ns (work->dev,NULL , &work->target, &mcaddr,NULL ,NULL ); +ndisc_send_ns (work->dev,NULL , &work->target, &mcaddr,NULL );dev_put (work->dev);kfree (work); }
Patches currently in stable-queue which might be from [email protected] are
queue-4.3/revert-ipv6-ndisc-inherit-metadata-dst-when-creating-ndisc-requests.patch