cvs commit: src/sys/netinet6 nd6.c

[John Hay]

On Sat, Sep 30, 2006 at 11:33:35PM +0900, Hajimu UMEMOTO wrote:
> jhay>   FreeBSD src repository
> 
> jhay>   Modified files:
> jhay>     sys/netinet6         nd6.c 
> jhay>   Log:
> jhay>   My previous commit broke "route add -inet6 <network_addr> -interface gif0".
> jhay>   Fix that by excluding point-to-point interfaces.
>   
> > -		} else if (req == RTM_ADD && SDL(gate)->sdl_alen == 0) {
> > +		} else if (req == RTM_ADD && SDL(gate)->sdl_alen == 0 &&
> > +		    (ifp->if_flags & (IFF_POINTOPOINT | IFF_LOOPBACK)) == 0) {
> > 			ln->ln_state = ND6_LLINFO_INCOMPLETE;
> 
> Umm, it seems still there is a problem.  We are using -interface not
> only for point-to-point interface and loopback.
> I'm not sure but can we check `(rt->rt_flags & RTF_HOST) != 0', here?

That would work better yes. Is there a reason you feel unsure about it?
Or is it just a case of not having tested it?

John
-- 
John Hay -- John.Hay at meraka.csir.co.za / jhay at FreeBSD.org