Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 08 Dec 2003 17:18:49 -0500
From:      "Brian F. Feldman" <green@FreeBSD.org>
To:        Hajimu UMEMOTO <ume@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/netinet6 nd6.c 
Message-ID:  <200312082218.hB8MInl1016806@green.bikeshed.org>
In-Reply-To: Message from Hajimu UMEMOTO <ume@FreeBSD.org>  <200312081159.hB8BxL4O068060@repoman.freebsd.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
Hajimu UMEMOTO <ume@FreeBSD.org> wrote:
> ume         2003/12/08 03:59:21 PST
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/netinet6         nd6.c 
>   Log:
>   - changed the logic in nd6_is_addr_neighbor(); check on-link prefixes
>     (not interface addresses) to see if a given address is on-link.
>   - skip offlink prefixes in neighbor determination in nd6_is_addr_neighbor.
>   - in nd6_is_addr_neighbor, regarded every address as on-link when the
>     default router list is empty.  otherwise, we'd not be able make a neighbor
>     cache for the address.
>     this algorithm is applied to hosts only.
>   - in nd6_is_addr_neighbor, check if the default interface is equal to
>     the interface in question in addition to check if the default router
>     list is empty.
>   
>   Obtained from:  KAME

Does this have any bearing on the recursion that I (worked around|fixed) 
because it was locking the same route twice and crashing?

--- nd6.c       21 Nov 2003 20:09:00 -0000      1.1
+++ nd6.c       8 Dec 2003 17:17:10 -0000
@@ -1095,7 +1095,8 @@
 
        if (req == RTM_RESOLVE &&
            (nd6_need_cache(ifp) == 0 || /* stf case */
-            !nd6_is_addr_neighbor((struct sockaddr_in6 *)rt_key(rt), ifp))) {
+           ((!(rt->rt_flags & RTF_WASCLONED) || rt->rt_flags & RTF_LLINFO) &&
+           !nd6_is_addr_neighbor((struct sockaddr_in6 *)rt_key(rt), ifp)))) {
                /*
                 * FreeBSD and BSD/OS often make a cloned host route based
                 * on a less-specific route (e.g. the default route).



-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green@FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200312082218.hB8MInl1016806>