ping6: fix sequence numbers (missed ntoh) and ttl display.

(apparently some, eh, clever libc guy decided that
*CHANGING* IPV6_HOPLIMIT value in libc header is a nifty idea...)

diff --git a/networking/ping.c b/networking/ping.c
index 8b49df1bd29fc3438b32fe7c463153ec03fd341f..de97d7e7fa9794f3d2ac3cd76e82a6acac34ccac 100644 (file)
--- a/networking/ping.c
+++ b/networking/ping.c
@@ -12,23 +12,11 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <sys/param.h>
-#include <sys/socket.h>
-#include <sys/file.h>
-#include <sys/times.h>
-#include <signal.h>
-
-#include <netinet/in.h>
-#include <netinet/ip.h>
+//#include <netinet/in.h>
+//#include <netinet/ip.h>
 #include <netinet/ip_icmp.h>
-#include <arpa/inet.h>
-#include <netdb.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <errno.h>
-#include <unistd.h>
-#include <string.h>
-#include <stdlib.h>
+//#include <arpa/inet.h>
+//#include <netdb.h>
 #include "busybox.h"
 
 enum {
@@ -120,9 +108,8 @@ static void ping(const char *host)
                c = recvfrom(pingsock, packet, sizeof(packet), 0,
                                (struct sockaddr *) &from, &fromlen);
                if (c < 0) {
-                       if (errno == EINTR)
-                               continue;
-                       bb_perror_msg("recvfrom");
+                       if (errno != EINTR)
+                               bb_perror_msg("recvfrom");
                        continue;
                }
                if (c >= 76) {                  /* ip + icmp */
@@ -135,7 +122,6 @@ static void ping(const char *host)
        }
        if (ENABLE_FEATURE_CLEAN_UP) close(pingsock);
        printf("%s is alive!\n", hostname);
-       return;
 }
 
 int ping_main(int argc, char **argv)
@@ -231,7 +217,7 @@ static void sendping(int junk)
 
        if (i < 0)
                bb_perror_msg_and_die("sendto");
-       else if ((size_t)i != sizeof(packet))
+       if ((size_t)i != sizeof(packet))
                bb_error_msg_and_die("ping wrote %d chars; %d expected", i,
                           (int)sizeof(packet));
 
@@ -328,7 +314,8 @@ static void unpack(char *buf, int sz, struct sockaddr_in *from)
        } else
                if (icmppkt->icmp_type != ICMP_ECHO)
                        bb_error_msg("warning: got ICMP %d (%s)",
-                                       icmppkt->icmp_type, icmp_type_name(icmppkt->icmp_type));
+                                       icmppkt->icmp_type,
+                                       icmp_type_name(icmppkt->icmp_type));
        fflush(stdout);
 }
 
@@ -380,11 +367,11 @@ static void ping(const char *host)
                socklen_t fromlen = (socklen_t) sizeof(from);
                int c;
 
-               if ((c = recvfrom(pingsock, packet, sizeof(packet), 0,
-                                                 (struct sockaddr *) &from, &fromlen)) < 0) {
-                       if (errno == EINTR)
-                               continue;
-                       bb_perror_msg("recvfrom");
+               c = recvfrom(pingsock, packet, sizeof(packet), 0,
+                               (struct sockaddr *) &from, &fromlen);
+               if (c < 0) {
+                       if (errno != EINTR)
+                               bb_perror_msg("recvfrom");
                        continue;
                }
                unpack(packet, c, &from);

diff --git a/networking/ping6.c b/networking/ping6.c
index c0b31c75265a2384d52c308a40e97b14c6c01720..81e16e2f024f4def396cabed6e6e2e21846cd213 100644 (file)
--- a/networking/ping6.c
+++ b/networking/ping6.c
@@ -22,27 +22,21 @@
  * The code was modified by Bart Visscher <magick@linux-fan.com>
  */
 
-#include <sys/param.h>
-#include <sys/socket.h>
-#include <sys/file.h>
-#include <sys/times.h>
-#include <signal.h>
-
-#include <netinet/in.h>
-#include <netinet/ip6.h>
+//#include <netinet/in.h>
+//#include <netinet/ip6.h>
 #include <netinet/icmp6.h>
-#include <arpa/inet.h>
+//#include <arpa/inet.h>
 #include <net/if.h>
-#include <netdb.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <errno.h>
-#include <unistd.h>
-#include <string.h>
-#include <stdlib.h>
-#include <stddef.h>                            /* offsetof */
+//#include <netdb.h>
 #include "busybox.h"
 
+/* I see RENUMBERED constants in bits/in.h - !!?
+ * What a fuck is going on with libc? Is it a glibc joke? */
+#ifdef IPV6_2292HOPLIMIT
+#undef IPV6_HOPLIMIT
+#define IPV6_HOPLIMIT IPV6_2292HOPLIMIT
+#endif
+
 enum {
        DEFDATALEN = 56,
        MAXIPLEN = 60,
@@ -94,7 +88,7 @@ static void ping(const char *host)
        c = sendto(pingsock, packet, DEFDATALEN + sizeof (struct icmp6_hdr), 0,
                           (struct sockaddr *) &pingaddr, sizeof(struct sockaddr_in6));
 
-       if (c < 0 || c != sizeof(packet)) {
+       if (c < 0) {
                if (ENABLE_FEATURE_CLEAN_UP) close(pingsock);
                bb_perror_msg_and_die("sendto");
        }
@@ -109,9 +103,8 @@ static void ping(const char *host)
                c = recvfrom(pingsock, packet, sizeof(packet), 0,
                                (struct sockaddr *) &from, &fromlen);
                if (c < 0) {
-                       if (errno == EINTR)
-                               continue;
-                       bb_perror_msg("recvfrom");
+                       if (errno != EINTR)
+                               bb_perror_msg("recvfrom");
                        continue;
                }
                if (c >= 8) {                   /* icmp6_hdr */
@@ -122,7 +115,6 @@ static void ping(const char *host)
        }
        if (ENABLE_FEATURE_CLEAN_UP) close(pingsock);
        printf("%s is alive!\n", h->h_name);
-       return;
 }
 
 int ping6_main(int argc, char **argv)
@@ -218,7 +210,7 @@ static void sendping(int junk)
 
        if (i < 0)
                bb_perror_msg_and_die("sendto");
-       else if ((size_t)i != sizeof(packet))
+       if ((size_t)i != sizeof(packet))
                bb_error_msg_and_die("ping wrote %d chars; %d expected", i,
                           (int)sizeof(packet));
 
@@ -246,16 +238,16 @@ static void sendping(int junk)
 static char *icmp6_type_name(int id)
 {
        switch (id) {
-       case ICMP6_DST_UNREACH:                         return "Destination Unreachable";
-       case ICMP6_PACKET_TOO_BIG:                      return "Packet too big";
-       case ICMP6_TIME_EXCEEDED:                       return "Time Exceeded";
-       case ICMP6_PARAM_PROB:                          return "Parameter Problem";
-       case ICMP6_ECHO_REPLY:                          return "Echo Reply";
-       case ICMP6_ECHO_REQUEST:                        return "Echo Request";
-       case MLD_LISTENER_QUERY:                        return "Listener Query";
-       case MLD_LISTENER_REPORT:                       return "Listener Report";
-       case MLD_LISTENER_REDUCTION:            return "Listener Reduction";
-       default:                                                        return "unknown ICMP type";
+       case ICMP6_DST_UNREACH:         return "Destination Unreachable";
+       case ICMP6_PACKET_TOO_BIG:      return "Packet too big";
+       case ICMP6_TIME_EXCEEDED:       return "Time Exceeded";
+       case ICMP6_PARAM_PROB:          return "Parameter Problem";
+       case ICMP6_ECHO_REPLY:          return "Echo Reply";
+       case ICMP6_ECHO_REQUEST:        return "Echo Request";
+       case MLD_LISTENER_QUERY:        return "Listener Query";
+       case MLD_LISTENER_REPORT:       return "Listener Report";
+       case MLD_LISTENER_REDUCTION:    return "Listener Reduction";
+       default:                        return "unknown ICMP type";
        }
 }
 
@@ -309,7 +301,7 @@ static void unpack(char *packet, int sz, struct sockaddr_in6 *from, int hoplimit
                printf("%d bytes from %s: icmp6_seq=%u", sz,
                           inet_ntop(AF_INET6, &pingaddr.sin6_addr,
                                                 buf, sizeof(buf)),
-                          icmppkt->icmp6_seq);
+                          ntohs(icmppkt->icmp6_seq));
                printf(" ttl=%d time=%lu.%lu ms", hoplimit,
                           triptime / 10, triptime % 10);
                if (dupflag)
@@ -361,14 +353,16 @@ static void ping(const char *host)
        setsockopt_broadcast(pingsock);
 
        /* set recv buf for broadcast pings */
-       sockopt = 48 * 1024;
+       sockopt = 48 * 1024; /* explain why 48k? */
        setsockopt(pingsock, SOL_SOCKET, SO_RCVBUF, (char *) &sockopt,
                           sizeof(sockopt));
 
+       sockopt = 2; /* iputils-ss020927 does this */
        sockopt = offsetof(struct icmp6_hdr, icmp6_cksum);
        setsockopt(pingsock, SOL_RAW, IPV6_CHECKSUM, (char *) &sockopt,
                           sizeof(sockopt));
 
+       /* request ttl info to be returned in ancillary data */
        sockopt = 1;
        setsockopt(pingsock, SOL_IPV6, IPV6_HOPLIMIT, (char *) &sockopt,
                           sizeof(sockopt));
@@ -377,10 +371,10 @@ static void ping(const char *host)
                pingaddr.sin6_scope_id = if_index;
 
        printf("PING %s (%s): %d data bytes\n",
-                  hostent->h_name,
-                  inet_ntop(AF_INET6, &pingaddr.sin6_addr,
+                       hostent->h_name,
+                       inet_ntop(AF_INET6, &pingaddr.sin6_addr,
                        buf, sizeof(buf)),
-                  datalen);
+                       datalen);
 
        signal(SIGINT, pingstats);
 
@@ -397,21 +391,23 @@ static void ping(const char *host)
        iov.iov_len = sizeof(packet);
        while (1) {
                int c;
-               struct cmsghdr *cmsgptr = NULL;
+               struct cmsghdr *mp;
                int hoplimit = -1;
                msg.msg_controllen = sizeof(control_buf);
 
-               if ((c = recvmsg(pingsock, &msg, 0)) < 0) {
-                       if (errno == EINTR)
-                               continue;
-                       bb_perror_msg("recvfrom");
+               c = recvmsg(pingsock, &msg, 0);
+               if (c < 0) {
+                       if (errno != EINTR)
+                               bb_perror_msg("recvfrom");
                        continue;
                }
-               for (cmsgptr = CMSG_FIRSTHDR(&msg); cmsgptr != NULL;
-                        cmsgptr = CMSG_NXTHDR(&msg, cmsgptr)) {
-                       if (cmsgptr->cmsg_level == SOL_IPV6 &&
-                               cmsgptr->cmsg_type == IPV6_HOPLIMIT ) {
-                               hoplimit = *(int*)CMSG_DATA(cmsgptr);
+               for (mp = CMSG_FIRSTHDR(&msg); mp; mp = CMSG_NXTHDR(&msg, mp)) {
+                       if (mp->cmsg_level == SOL_IPV6
+                        && mp->cmsg_type == IPV6_HOPLIMIT
+                        /* don't check len - we trust the kernel: */
+                        /* && mp->cmsg_len >= CMSG_LEN(sizeof(int)) */
+                       ) {
+                               hoplimit = *(int*)CMSG_DATA(mp);
                        }
                }
                unpack(packet, c, &from, hoplimit);