Commit ba9669a6 authored by Eli Cooper's avatar Eli Cooper Committed by Kleber Sacilotto de Souza

ip6_tunnel: disable dst caching if tunnel is dual-stack

BugLink: http://bugs.launchpad.net/bugs/1745266

[ Upstream commit 23263ec8 ]

When an ip6_tunnel is in mode 'any', where the transport layer
protocol can be either 4 or 41, dst_cache must be disabled.

This is because xfrm policies might apply to only one of the two
protocols. Caching dst would cause xfrm policies for one protocol
incorrectly used for the other.
Signed-off-by: default avatarEli Cooper <elicooper@gmx.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
parent 48f545ce
...@@ -1083,10 +1083,11 @@ static int ip6_tnl_xmit2(struct sk_buff *skb, ...@@ -1083,10 +1083,11 @@ static int ip6_tnl_xmit2(struct sk_buff *skb,
memcpy(&fl6->daddr, addr6, sizeof(fl6->daddr)); memcpy(&fl6->daddr, addr6, sizeof(fl6->daddr));
neigh_release(neigh); neigh_release(neigh);
} }
} else if (!(t->parms.flags & } else if (t->parms.proto != 0 && !(t->parms.flags &
(IP6_TNL_F_USE_ORIG_TCLASS | IP6_TNL_F_USE_ORIG_FWMARK))) { (IP6_TNL_F_USE_ORIG_TCLASS |
/* enable the cache only only if the routing decision does IP6_TNL_F_USE_ORIG_FWMARK))) {
* not depend on the current inner header value /* enable the cache only if neither the outer protocol nor the
* routing decision depends on the current inner header value
*/ */
use_cache = true; use_cache = true;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment