aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--Changelog2
-rw-r--r--docs/busybox.pod3
-rw-r--r--networking/ping.c57
-rw-r--r--ping.c57
4 files changed, 68 insertions, 51 deletions
diff --git a/Changelog b/Changelog
index 50b28cf62..493401d66 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 3cd45f7bd..7a04b8f10 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 <ezanardi@ull.es>
=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 14a56cd55..0cfe06229 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 <tausq@debian.org>
@@ -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 14a56cd55..0cfe06229 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 <tausq@debian.org>
@@ -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.