From 0024abcbbc633f4493b97c4b344ed636ea176f3c Mon Sep 17 00:00:00 2001 From: Pavel Roskin Date: Wed, 7 Jun 2000 20:38:15 +0000 Subject: [PATCH] Implemented "ping -s", fixed error messages and argument parsing --- Changelog | 2 ++ docs/busybox.pod | 3 ++- networking/ping.c | 57 +++++++++++++++++++++++++++++++------------------------ ping.c | 57 +++++++++++++++++++++++++++++++------------------------ 4 files changed, 68 insertions(+), 51 deletions(-) diff --git a/Changelog b/Changelog index 50b28cf..493401d 100644 --- a/Changelog +++ b/Changelog @@ -63,6 +63,8 @@ * Fixed 'swapon -a' and 'swapoff -a', which were broken. * Fixed 'mount -a' so it works as expected. * Implemented 'ls -R' (enabled by enabling BB_FEATURE_LS_RECURSIVE) + * Implemented "ping -s", fixed error messages and argument parsing - + Pavel Roskin * More doc updates diff --git a/docs/busybox.pod b/docs/busybox.pod index 3cd45f7..7a04b8f 100644 --- a/docs/busybox.pod +++ b/docs/busybox.pod @@ -1250,6 +1250,7 @@ Send ICMP ECHO_REQUEST packets to network hosts. Options: -c COUNT Send only COUNT pings. + -s SIZE Send SIZE data bytes in packets (default=56). -q Quiet mode, only displays output at start and when finished. Example: @@ -1948,4 +1949,4 @@ Enrique Zanardi =cut -# $Id: busybox.pod,v 1.36 2000/06/07 17:28:53 andersen Exp $ +# $Id: busybox.pod,v 1.37 2000/06/07 20:38:15 proski Exp $ diff --git a/networking/ping.c b/networking/ping.c index 14a56cd..0cfe062 100644 --- a/networking/ping.c +++ b/networking/ping.c @@ -1,6 +1,6 @@ /* vi: set sw=4 ts=4: */ /* - * $Id: ping.c,v 1.15 2000/05/12 19:41:47 erik Exp $ + * $Id: ping.c,v 1.16 2000/06/07 20:38:15 proski Exp $ * Mini ping implementation for busybox * * Copyright (C) 1999 by Randolph Chung @@ -64,6 +64,8 @@ #define CLR(bit) (A(bit) &= (~B(bit))) #define TST(bit) (A(bit) & B(bit)) +static void ping(const char *host); + /* common routines */ static int in_cksum(unsigned short *buf, int sz) { @@ -104,7 +106,7 @@ static void noresp(int ign) exit(0); } -static int ping(const char *host) +static void ping(const char *host) { struct hostent *h; struct sockaddr_in pingaddr; @@ -113,7 +115,7 @@ static int ping(const char *host) char packet[DEFDATALEN + MAXIPLEN + MAXICMPLEN]; if ((pingsock = socket(AF_INET, SOCK_RAW, 1)) < 0) { /* 1 == ICMP */ - perror("ping"); + perror("ping: creating a raw socket"); exit(1); } @@ -140,7 +142,7 @@ static int ping(const char *host) if (c < 0 || c != sizeof(packet)) { if (c < 0) - perror("ping"); + perror("ping: sendto"); fprintf(stderr, "ping: write incomplete\n"); exit(1); } @@ -156,7 +158,7 @@ static int ping(const char *host) (struct sockaddr *) &from, &fromlen)) < 0) { if (errno == EINTR) continue; - perror("ping"); + perror("ping: recvfrom"); continue; } if (c >= 76) { /* ip + icmp */ @@ -168,7 +170,7 @@ static int ping(const char *host) } } printf("%s is alive!\n", hostname); - return (TRUE); + return; } extern int ping_main(int argc, char **argv) @@ -181,13 +183,14 @@ extern int ping_main(int argc, char **argv) exit(TRUE); } -#else +#else /* ! BB_SIMPLE_PING */ /* full(er) version */ static const char *ping_usage = "ping [OPTION]... host\n" #ifndef BB_FEATURE_TRIVIAL_HELP "\nSend ICMP ECHO_REQUEST packets to network hosts.\n\n" "Options:\n" "\t-c COUNT\tSend only COUNT pings.\n" + "\t-s SIZE\t\tSend SIZE data bytes in packets (default=56).\n" "\t-q\t\tQuiet mode, only displays output at start\n" "\t\t\tand when finished.\n" #endif @@ -196,6 +199,7 @@ static const char *ping_usage = "ping [OPTION]... host\n" static char *hostname = NULL; static struct sockaddr_in pingaddr; static int pingsock = -1; +static int datalen = DEFDATALEN; static long ntransmitted = 0, nreceived = 0, nrepeats = 0, pingcount = 0; static int myid = 0, options = 0; @@ -206,8 +210,6 @@ static void sendping(int); static void pingstats(int); static void unpack(char *, int, struct sockaddr_in *); -static void ping(char *); - /**************************************************************************/ static void pingstats(int ign) @@ -234,7 +236,7 @@ static void sendping(int ign) { struct icmp *pkt; int i; - char packet[DEFDATALEN + 8]; + char packet[datalen + 8]; pkt = (struct icmp *) packet; @@ -251,13 +253,11 @@ static void sendping(int ign) i = sendto(pingsock, packet, sizeof(packet), 0, (struct sockaddr *) &pingaddr, sizeof(struct sockaddr_in)); - if (i < 0 || i != sizeof(packet)) { - if (i < 0) - perror("ping"); - fprintf(stderr, "ping wrote %d chars; %d expected\n", i, - (int)sizeof(packet)); - exit(1); - } + if (i < 0) + fatalError("ping: sendto: %s\n", strerror(errno)); + else if (i != sizeof(packet)) + fatalError("ping wrote %d chars; %d expected\n", i, + (int)sizeof(packet)); signal(SIGALRM, sendping); if (pingcount == 0 || ntransmitted < pingcount) { /* schedule next in 1s */ @@ -303,7 +303,7 @@ static void unpack(char *buf, int sz, struct sockaddr_in *from) iphdr = (struct iphdr *) buf; hlen = iphdr->ihl << 2; /* discard if too short */ - if (sz < (DEFDATALEN + ICMP_MINLEN)) + if (sz < (datalen + ICMP_MINLEN)) return; sz -= hlen; @@ -356,12 +356,12 @@ static void unpack(char *buf, int sz, struct sockaddr_in *from) icmppkt->icmp_type, icmp_type_name (icmppkt->icmp_type)); } -static void ping(char *host) +static void ping(const char *host) { struct protoent *proto; struct hostent *h; char buf[MAXHOSTNAMELEN]; - char packet[DEFDATALEN + MAXIPLEN + MAXICMPLEN]; + char packet[datalen + MAXIPLEN + MAXICMPLEN]; int sockopt; proto = getprotobyname("icmp"); @@ -372,7 +372,7 @@ static void ping(char *host) if (errno == EPERM) { fprintf(stderr, "ping: permission denied. (are you root?)\n"); } else { - perror("ping"); + perror("ping: creating a raw socket"); } exit(1); } @@ -412,7 +412,7 @@ static void ping(char *host) printf("PING %s (%s): %d data bytes\n", hostname, inet_ntoa(*(struct in_addr *) &pingaddr.sin_addr.s_addr), - DEFDATALEN); + datalen); signal(SIGINT, pingstats); @@ -429,7 +429,7 @@ static void ping(char *host) (struct sockaddr *) &from, &fromlen)) < 0) { if (errno == EINTR) continue; - perror("ping"); + perror("ping: recvfrom"); continue; } unpack(packet, c, &from); @@ -455,10 +455,17 @@ extern int ping_main(int argc, char **argv) options |= O_QUIET; break; case 'c': - argc--; + if (--argc <= 0) + usage(ping_usage); argv++; pingcount = atoi(*argv); break; + case 's': + if (--argc <= 0) + usage(ping_usage); + argv++; + datalen = atoi(*argv); + break; default: usage(ping_usage); } @@ -472,7 +479,7 @@ extern int ping_main(int argc, char **argv) ping(*argv); exit(TRUE); } -#endif +#endif /* ! BB_SIMPLE_PING */ /* * Copyright (c) 1989 The Regents of the University of California. diff --git a/ping.c b/ping.c index 14a56cd..0cfe062 100644 --- a/ping.c +++ b/ping.c @@ -1,6 +1,6 @@ /* vi: set sw=4 ts=4: */ /* - * $Id: ping.c,v 1.15 2000/05/12 19:41:47 erik Exp $ + * $Id: ping.c,v 1.16 2000/06/07 20:38:15 proski Exp $ * Mini ping implementation for busybox * * Copyright (C) 1999 by Randolph Chung @@ -64,6 +64,8 @@ #define CLR(bit) (A(bit) &= (~B(bit))) #define TST(bit) (A(bit) & B(bit)) +static void ping(const char *host); + /* common routines */ static int in_cksum(unsigned short *buf, int sz) { @@ -104,7 +106,7 @@ static void noresp(int ign) exit(0); } -static int ping(const char *host) +static void ping(const char *host) { struct hostent *h; struct sockaddr_in pingaddr; @@ -113,7 +115,7 @@ static int ping(const char *host) char packet[DEFDATALEN + MAXIPLEN + MAXICMPLEN]; if ((pingsock = socket(AF_INET, SOCK_RAW, 1)) < 0) { /* 1 == ICMP */ - perror("ping"); + perror("ping: creating a raw socket"); exit(1); } @@ -140,7 +142,7 @@ static int ping(const char *host) if (c < 0 || c != sizeof(packet)) { if (c < 0) - perror("ping"); + perror("ping: sendto"); fprintf(stderr, "ping: write incomplete\n"); exit(1); } @@ -156,7 +158,7 @@ static int ping(const char *host) (struct sockaddr *) &from, &fromlen)) < 0) { if (errno == EINTR) continue; - perror("ping"); + perror("ping: recvfrom"); continue; } if (c >= 76) { /* ip + icmp */ @@ -168,7 +170,7 @@ static int ping(const char *host) } } printf("%s is alive!\n", hostname); - return (TRUE); + return; } extern int ping_main(int argc, char **argv) @@ -181,13 +183,14 @@ extern int ping_main(int argc, char **argv) exit(TRUE); } -#else +#else /* ! BB_SIMPLE_PING */ /* full(er) version */ static const char *ping_usage = "ping [OPTION]... host\n" #ifndef BB_FEATURE_TRIVIAL_HELP "\nSend ICMP ECHO_REQUEST packets to network hosts.\n\n" "Options:\n" "\t-c COUNT\tSend only COUNT pings.\n" + "\t-s SIZE\t\tSend SIZE data bytes in packets (default=56).\n" "\t-q\t\tQuiet mode, only displays output at start\n" "\t\t\tand when finished.\n" #endif @@ -196,6 +199,7 @@ static const char *ping_usage = "ping [OPTION]... host\n" static char *hostname = NULL; static struct sockaddr_in pingaddr; static int pingsock = -1; +static int datalen = DEFDATALEN; static long ntransmitted = 0, nreceived = 0, nrepeats = 0, pingcount = 0; static int myid = 0, options = 0; @@ -206,8 +210,6 @@ static void sendping(int); static void pingstats(int); static void unpack(char *, int, struct sockaddr_in *); -static void ping(char *); - /**************************************************************************/ static void pingstats(int ign) @@ -234,7 +236,7 @@ static void sendping(int ign) { struct icmp *pkt; int i; - char packet[DEFDATALEN + 8]; + char packet[datalen + 8]; pkt = (struct icmp *) packet; @@ -251,13 +253,11 @@ static void sendping(int ign) i = sendto(pingsock, packet, sizeof(packet), 0, (struct sockaddr *) &pingaddr, sizeof(struct sockaddr_in)); - if (i < 0 || i != sizeof(packet)) { - if (i < 0) - perror("ping"); - fprintf(stderr, "ping wrote %d chars; %d expected\n", i, - (int)sizeof(packet)); - exit(1); - } + if (i < 0) + fatalError("ping: sendto: %s\n", strerror(errno)); + else if (i != sizeof(packet)) + fatalError("ping wrote %d chars; %d expected\n", i, + (int)sizeof(packet)); signal(SIGALRM, sendping); if (pingcount == 0 || ntransmitted < pingcount) { /* schedule next in 1s */ @@ -303,7 +303,7 @@ static void unpack(char *buf, int sz, struct sockaddr_in *from) iphdr = (struct iphdr *) buf; hlen = iphdr->ihl << 2; /* discard if too short */ - if (sz < (DEFDATALEN + ICMP_MINLEN)) + if (sz < (datalen + ICMP_MINLEN)) return; sz -= hlen; @@ -356,12 +356,12 @@ static void unpack(char *buf, int sz, struct sockaddr_in *from) icmppkt->icmp_type, icmp_type_name (icmppkt->icmp_type)); } -static void ping(char *host) +static void ping(const char *host) { struct protoent *proto; struct hostent *h; char buf[MAXHOSTNAMELEN]; - char packet[DEFDATALEN + MAXIPLEN + MAXICMPLEN]; + char packet[datalen + MAXIPLEN + MAXICMPLEN]; int sockopt; proto = getprotobyname("icmp"); @@ -372,7 +372,7 @@ static void ping(char *host) if (errno == EPERM) { fprintf(stderr, "ping: permission denied. (are you root?)\n"); } else { - perror("ping"); + perror("ping: creating a raw socket"); } exit(1); } @@ -412,7 +412,7 @@ static void ping(char *host) printf("PING %s (%s): %d data bytes\n", hostname, inet_ntoa(*(struct in_addr *) &pingaddr.sin_addr.s_addr), - DEFDATALEN); + datalen); signal(SIGINT, pingstats); @@ -429,7 +429,7 @@ static void ping(char *host) (struct sockaddr *) &from, &fromlen)) < 0) { if (errno == EINTR) continue; - perror("ping"); + perror("ping: recvfrom"); continue; } unpack(packet, c, &from); @@ -455,10 +455,17 @@ extern int ping_main(int argc, char **argv) options |= O_QUIET; break; case 'c': - argc--; + if (--argc <= 0) + usage(ping_usage); argv++; pingcount = atoi(*argv); break; + case 's': + if (--argc <= 0) + usage(ping_usage); + argv++; + datalen = atoi(*argv); + break; default: usage(ping_usage); } @@ -472,7 +479,7 @@ extern int ping_main(int argc, char **argv) ping(*argv); exit(TRUE); } -#endif +#endif /* ! BB_SIMPLE_PING */ /* * Copyright (c) 1989 The Regents of the University of California. -- 2.7.4