Module Name: src Committed By: christos Date: Sat Jun 8 02:42:56 UTC 2013
Modified Files: src/sys/net: route.c Log Message: PR/44032: Proxy entries stopped working with pppd. The issue here is that the route entry was added, but the RTF_LLINFO bit was not set, making arp -a not showing the entry, but netstat -rn -f inet showing it with the missing L bit. The order of resolution in ifa_ifwithroute() is that if a destination address is found, then the interface chosen for the route is that of the destination. This does not work for link-level addresses since the ppp interface does not arp (uses link_rtrequest, not arp_rtrequest), so the bit is never set. The easy solution here is to check that the gateway is a link address, and use the interface which we chose for the link address as opposed to the interface that routes to the destination. This restores the previous behavior, but is it correct? To generate a diff of this commit: cvs rdiff -u -r1.126 -r1.127 src/sys/net/route.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/net/route.c diff -u src/sys/net/route.c:1.126 src/sys/net/route.c:1.127 --- src/sys/net/route.c:1.126 Mon Jan 30 15:01:08 2012 +++ src/sys/net/route.c Fri Jun 7 22:42:56 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: route.c,v 1.126 2012/01/30 20:01:08 christos Exp $ */ +/* $NetBSD: route.c,v 1.127 2013/06/08 02:42:56 christos Exp $ */ /*- * Copyright (c) 1998, 2008 The NetBSD Foundation, Inc. @@ -93,7 +93,7 @@ #include "opt_route.h" #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.126 2012/01/30 20:01:08 christos Exp $"); +__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.127 2013/06/08 02:42:56 christos Exp $"); #include <sys/param.h> #include <sys/kmem.h> @@ -594,7 +594,7 @@ ifa_ifwithroute(int flags, const struct * we can use the local address. */ ifa = NULL; - if (flags & RTF_HOST) + if ((flags & RTF_HOST) && gateway->sa_family != AF_LINK) ifa = ifa_ifwithdstaddr(dst); if (ifa == NULL) ifa = ifa_ifwithaddr(gateway); @@ -619,7 +619,7 @@ ifa_ifwithroute(int flags, const struct if (ifa->ifa_addr->sa_family != dst->sa_family) { struct ifaddr *oifa = ifa; ifa = ifaof_ifpforaddr(dst, ifa->ifa_ifp); - if (ifa == 0) + if (ifa == NULL) ifa = oifa; } return ifa;