svn commit: r287920 - head/sbin/route
Ryan Stone
rstone at FreeBSD.org
Thu Sep 17 16:56:50 UTC 2015
Author: rstone
Date: Thu Sep 17 16:56:49 2015
New Revision: 287920
URL: https://svnweb.freebsd.org/changeset/base/287920
Log:
Fix /sbin/route to never look up (invalid) interface names through DNS
/sbin/route has a bug where if it is passed an interface name that does
not exist, it falls through and winds up interpreting it as a hostname.
It fails out eventually, but on a system where DNS lookup is broken you
can end up waiting for up to 60 seconds waiting for the DNS lookup to
timeout. I'm not quite sure what happens if the DNS lookup somehow
succeeds but I doubt that can end well.
Reviewed by: markj, cem
MFC after: 2 weeks
Sponsored by: EMC/Isilon Storage Division
Modified:
head/sbin/route/route.c
Modified: head/sbin/route/route.c
==============================================================================
--- head/sbin/route/route.c Thu Sep 17 16:38:32 2015 (r287919)
+++ head/sbin/route/route.c Thu Sep 17 16:56:49 2015 (r287920)
@@ -1222,6 +1222,9 @@ getaddr(int idx, char *str, struct hoste
freeifaddrs(ifap);
if (sdl != NULL)
return(1);
+ else
+ errx(EX_DATAERR,
+ "interface '%s' does not exist", str);
}
break;
case RTAX_IFP:
More information about the svn-src-head
mailing list