Commit d76e60a5 authored by David S. Miller's avatar David S. Miller

[IPV6]: Fix some code/comment formatting in ip6_dst_output().

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 06ef921d
...@@ -733,28 +733,29 @@ int ip6_dst_lookup(struct sock *sk, struct dst_entry **dst, struct flowi *fl) ...@@ -733,28 +733,29 @@ int ip6_dst_lookup(struct sock *sk, struct dst_entry **dst, struct flowi *fl)
if (*dst) { if (*dst) {
struct rt6_info *rt = (struct rt6_info*)*dst; struct rt6_info *rt = (struct rt6_info*)*dst;
/* Yes, checking route validity in not connected /* Yes, checking route validity in not connected
case is not very simple. Take into account, * case is not very simple. Take into account,
that we do not support routing by source, TOS, * that we do not support routing by source, TOS,
and MSG_DONTROUTE --ANK (980726) * and MSG_DONTROUTE --ANK (980726)
*
1. If route was host route, check that * 1. If route was host route, check that
cached destination is current. * cached destination is current.
If it is network route, we still may * If it is network route, we still may
check its validity using saved pointer * check its validity using saved pointer
to the last used address: daddr_cache. * to the last used address: daddr_cache.
We do not want to save whole address now, * We do not want to save whole address now,
(because main consumer of this service * (because main consumer of this service
is tcp, which has not this problem), * is tcp, which has not this problem),
so that the last trick works only on connected * so that the last trick works only on connected
sockets. * sockets.
2. oif also should be the same. * 2. oif also should be the same.
*/ */
if (((rt->rt6i_dst.plen != 128 || if (((rt->rt6i_dst.plen != 128 ||
!ipv6_addr_equal(&fl->fl6_dst, &rt->rt6i_dst.addr)) !ipv6_addr_equal(&fl->fl6_dst,
&rt->rt6i_dst.addr))
&& (np->daddr_cache == NULL || && (np->daddr_cache == NULL ||
!ipv6_addr_equal(&fl->fl6_dst, np->daddr_cache))) !ipv6_addr_equal(&fl->fl6_dst,
np->daddr_cache)))
|| (fl->oif && fl->oif != (*dst)->dev->ifindex)) { || (fl->oif && fl->oif != (*dst)->dev->ifindex)) {
dst_release(*dst); dst_release(*dst);
*dst = NULL; *dst = NULL;
......
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