netdev - ping6

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <fa4b41e8992b983d77dc2b57c72431e7.squirrel@charon>
Date:	Mon, 18 Jul 2011 14:23:33 -0600
From:	eric@...tus.bz
To:	netdev@...r.kernel.org
Subject: ping6

Hi,
I've found what I think is a bug in the ping6 tool. Without the -n option,
it puts the hostname inside the parenthesis instead of the resolved ip
address as demonstrated below:

ericat@...ra:~/iputils-s20101006$ ping6 ipv6.he.net
PING ipv6.he.net(ipv6.he.net) 56 data bytes
64 bytes from ipv6.he.net: icmp_seq=1 ttl=58 time=186 ms

I expect output like this:

ericat@...ra:~/iputils-s20101006$ sudo ./ping6 ipv6.he.net
Password:
PING ipv6.he.net(2001:470:0:64::2) 56 data bytes
64 bytes from ipv6.he.net: icmp_seq=1 ttl=58 time=187 ms

Here's a very simple patch to ping6.c for proper behaviour:

ericat@...ra:~/iputils-s20101006$ diff ping6.c.orig ping6.c
979c979
<       printf("PING %s(%s) ", hostname, pr_addr(&whereto.sin6_addr));
---
>       printf("PING %s(%s) ", hostname, pr_addr_n(&whereto.sin6_addr));

Cheers,
Eric Atkin


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html