svn commit: r233773 - head/usr.sbin/arp
Gleb Smirnoff
glebius at FreeBSD.org
Mon Apr 2 10:44:25 UTC 2012
Author: glebius
Date: Mon Apr 2 10:44:25 2012
New Revision: 233773
URL: http://svn.freebsd.org/changeset/base/233773
Log:
Historically arp(8) did a route lookup for the entry it is
about to add, and failed if it exist and had invalid data
link type.
Later on, in r201282, this check morphed to other code, but
message "proxy entry exists for non 802 device" still left,
and now it is printed in a case if route prefix found is
equal to current address being added. In other words, when
we are trying to add ARP entry for a network address. The
message is absolutely unrelated and disappointing in this
case.
I don't see anything bad with setting ARP entries for
network addresses. While useless in usual network,
in a /31 RFC3021 it may be necessary. This, remove this code.
Modified:
head/usr.sbin/arp/arp.c
Modified: head/usr.sbin/arp/arp.c
==============================================================================
--- head/usr.sbin/arp/arp.c Mon Apr 2 10:24:50 2012 (r233772)
+++ head/usr.sbin/arp/arp.c Mon Apr 2 10:44:25 2012 (r233773)
@@ -387,10 +387,6 @@ set(int argc, char **argv)
}
addr = (struct sockaddr_inarp *)(rtm + 1);
sdl = (struct sockaddr_dl *)(SA_SIZE(addr) + (char *)addr);
- if (addr->sin_addr.s_addr == dst->sin_addr.s_addr) {
- printf("set: proxy entry exists for non 802 device\n");
- return (1);
- }
if ((sdl->sdl_family != AF_LINK) ||
(rtm->rtm_flags & RTF_GATEWAY) ||
More information about the svn-src-head
mailing list