From d26245171514e85becdd350ee5574f7c886a5db0 Mon Sep 17 00:00:00 2001 From: Alan Somers Date: Thu, 26 Nov 2020 18:33:04 +0000 Subject: [PATCH] ping: add a ping6 hard link for backwards compatibility When invoked as "ping6", ping will now attempt to use ICMPv6 for hostnames that resolve both IPv4 and IPv6 addresses. Reviewed by: bz, manu MFC-With: r368045 Differential Revision: https://reviews.freebsd.org/D27384 --- ObsoleteFiles.inc | 2 -- rescue/rescue/Makefile | 1 + sbin/ping/Makefile | 1 + sbin/ping/main.c | 5 ++++- sbin/ping/ping.8 | 5 ++++- sbin/ping/tests/ping_test.sh | 14 ++++++++++++++ tools/build/mk/OptionalObsoleteFiles.inc | 2 ++ 7 files changed, 26 insertions(+), 4 deletions(-) diff --git a/ObsoleteFiles.inc b/ObsoleteFiles.inc index c69e030af1f..14313e57317 100644 --- a/ObsoleteFiles.inc +++ b/ObsoleteFiles.inc @@ -37,8 +37,6 @@ # done # 20201124: ping6(8) was merged into ping(8) -OLD_FILES+=sbin/ping6 -OLD_FILES+=rescue/ping6 OLD_FILES+=usr/lib/debug/sbin/ping6.debug OLD_FILES+=usr/share/man/man8/ping6.8.gz OLD_FILES+=usr/tests/sbin/ping6/Kyuafile diff --git a/rescue/rescue/Makefile b/rescue/rescue/Makefile index ae828a020e2..7352828a822 100644 --- a/rescue/rescue/Makefile +++ b/rescue/rescue/Makefile @@ -103,6 +103,7 @@ CRUNCH_PROGS_sbin+= ccdconfig .endif .if ${MK_INET6_SUPPORT} != "no" +CRUNCH_ALIAS_ping= ping6 CRUNCH_PROGS_sbin+= rtsol .endif diff --git a/sbin/ping/Makefile b/sbin/ping/Makefile index 437d2f6b1b5..c9cc63bbf74 100644 --- a/sbin/ping/Makefile +++ b/sbin/ping/Makefile @@ -15,6 +15,7 @@ LIBADD= m CFLAGS+= -DINET6 -DKAME_SCOPEID SRCS+= ping6.c LIBADD+= md +LINKS= ${BINDIR}/ping ${BINDIR}/ping6 .endif .if ${MK_DYNAMICROOT} == "no" diff --git a/sbin/ping/main.c b/sbin/ping/main.c index 13891e15da3..96150beec8b 100644 --- a/sbin/ping/main.c +++ b/sbin/ping/main.c @@ -66,7 +66,10 @@ main(int argc, char *argv[]) struct in6_addr a6; bool ipv6; - ipv6 = false; + if (strcmp(getprogname(), "ping6") == 0) + ipv6 = true; + else + ipv6 = false; #endif ipv4 = false; diff --git a/sbin/ping/ping.8 b/sbin/ping/ping.8 index c76a995b3a5..dd17725608e 100644 --- a/sbin/ping/ping.8 +++ b/sbin/ping/ping.8 @@ -28,7 +28,7 @@ .\" @(#)ping.8 8.2 (Berkeley) 12/11/93 .\" $FreeBSD$ .\" -.Dd November 24, 2020 +.Dd November 26, 2020 .Dt PING 8 .Os .Sh NAME @@ -141,6 +141,9 @@ the specific IP version can be requested by or .Fl 6 options, respectively. +For backwards-compatibility, ICMPv6 can also be selected by invoking the binary +as +.Nm ping6 . .Ss Options common to both IPv4 and IPv6 targets .Bl -tag -width indent .It Fl A diff --git a/sbin/ping/tests/ping_test.sh b/sbin/ping/tests/ping_test.sh index 5084b89d5a8..ed95594abbd 100644 --- a/sbin/ping/tests/ping_test.sh +++ b/sbin/ping/tests/ping_test.sh @@ -53,9 +53,23 @@ ping_6_c1_s8_t1_body() { check_ping_statistics std.out $(atf_get_srcdir)/ping_6_c1_s8_t1.out } +atf_test_case ping6_c1_s8_t1 +ping6_c1_s8_t1_head() { + atf_set "descr" "Use IPv6 when invoked as ping6" +} +ping6_c1_s8_t1_body() { + if ! getaddrinfo -f inet6 localhost 1>/dev/null 2>&1; then + atf_skip "IPv6 is not configured" + fi + atf_check -s exit:0 -o save:std.out -e empty \ + ping6 -c 1 -s 8 -t 1 localhost + check_ping_statistics std.out $(atf_get_srcdir)/ping_6_c1_s8_t1.out +} + atf_init_test_cases() { atf_add_test_case ping_c1_s56_t1 atf_add_test_case ping_6_c1_s8_t1 + atf_add_test_case ping6_c1_s8_t1 } check_ping_statistics() { diff --git a/tools/build/mk/OptionalObsoleteFiles.inc b/tools/build/mk/OptionalObsoleteFiles.inc index 658a049603a..69065a9c4a3 100644 --- a/tools/build/mk/OptionalObsoleteFiles.inc +++ b/tools/build/mk/OptionalObsoleteFiles.inc @@ -2583,6 +2583,7 @@ OLD_DIRS+=usr/share/i18n/csmapper/GB .endif .if ${MK_INET6} == no +OLD_FILES+=sbin/ping6 OLD_FILES+=sbin/rtsol OLD_FILES+=usr/sbin/ip6addrctl OLD_FILES+=usr/sbin/mld6query @@ -2611,6 +2612,7 @@ OLD_FILES+=usr/share/man/man8/traceroute6.8.gz .endif .if ${MK_INET6_SUPPORT} == no +OLD_FILES+=rescue/ping6 OLD_FILES+=rescue/rtsol .endif -- 2.45.0