aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--include/libbb.h13
-rw-r--r--include/usage.h2
-rw-r--r--libbb/xconnect.c50
-rw-r--r--networking/ftpgetput.c2
-rw-r--r--networking/nc.c103
-rw-r--r--networking/wget.c2
6 files changed, 95 insertions, 77 deletions
diff --git a/include/libbb.h b/include/libbb.h
index 07b1d1158..c088946d9 100644
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -307,6 +307,9 @@ typedef struct len_and_sockaddr {
#endif
};
} len_and_sockaddr;
+/* Create stream socket, and allocated suitable lsa
+ * (lsa of correct size and lsa->sa.sa_family (AF_INET/AF_INET6)) */
+int xsocket_stream(len_and_sockaddr **lsap);
/* Create server TCP socket bound to bindaddr:port. bindaddr can be NULL,
* numeric IP ("N.N.N.N") or numeric IPv6 address,
* and can have ":PORT" suffix (for IPv6 use "[X:X:...:X]:PORT").
@@ -324,12 +327,14 @@ extern int xconnect_stream(const len_and_sockaddr *lsa);
* (depending on host), but in theory nothing prevents e.g.
* UNIX socket address being returned, IPX sockaddr etc... */
extern len_and_sockaddr* host2sockaddr(const char *host, int port);
-/* assign sin[6]_port member if the socket is of corresponding type,
+/* Assign sin[6]_port member if the socket is of corresponding type,
* otherwise noop. Useful for ftp.
* NB: does NOT do htons() internally, just direct assignment. */
-extern void set_port(len_and_sockaddr *lsa, unsigned port);
-char* xmalloc_sockaddr2host(const struct sockaddr *sa, socklen_t salen);
-char* xmalloc_sockaddr2dotted(const struct sockaddr *sa, socklen_t salen);
+extern void set_nport(len_and_sockaddr *lsa, unsigned port);
+/* Retrieve sin[6]_port or return -1 for non-inet lsa's */
+extern int get_nport(len_and_sockaddr *lsa);
+extern char* xmalloc_sockaddr2host(const struct sockaddr *sa, socklen_t salen);
+extern char* xmalloc_sockaddr2dotted(const struct sockaddr *sa, socklen_t salen);
extern char *xstrdup(const char *s);
diff --git a/include/usage.h b/include/usage.h
index 4e5206558..0275df3f0 100644
--- a/include/usage.h
+++ b/include/usage.h
@@ -2249,7 +2249,7 @@
"\n -e Exec rest of command line after connect" \
"\n -i SECS Delay interval for lines sent" \
"\n -w SECS Timeout for connect" \
- "\n -f file Use file (ala /dev/ttyS0) instead of network" \
+ "\n -f FILE Use file (ala /dev/ttyS0) instead of network" \
) \
USE_NC_SERVER( \
"\n -l Listen mode, for inbound connects" \
diff --git a/libbb/xconnect.c b/libbb/xconnect.c
index f5a7e6dc8..65554b24e 100644
--- a/libbb/xconnect.c
+++ b/libbb/xconnect.c
@@ -32,7 +32,7 @@ void xconnect(int s, const struct sockaddr *s_addr, socklen_t addrlen)
}
}
-/* Return network byte ordered port number for a service.
+/* Return port number for a service.
* If "port" is a number use it as the port.
* If "port" is a name it is looked up in /etc/services, if it isnt found return
* default_port */
@@ -81,7 +81,21 @@ int xconnect_tcp_v4(struct sockaddr_in *s_addr)
/* "New" networking API */
-void set_port(len_and_sockaddr *lsa, unsigned port)
+int get_nport(len_and_sockaddr *lsa)
+{
+#if ENABLE_FEATURE_IPV6
+ if (lsa->sa.sa_family == AF_INET6) {
+ return lsa->sin6.sin6_port;
+ }
+#endif
+ if (lsa->sa.sa_family == AF_INET) {
+ return lsa->sin.sin_port;
+ }
+ return -1;
+ /* What? UNIX socket? IPX?? :) */
+}
+
+void set_nport(len_and_sockaddr *lsa, unsigned port)
{
#if ENABLE_FEATURE_IPV6
if (lsa->sa.sa_family == AF_INET6) {
@@ -146,7 +160,7 @@ static len_and_sockaddr* str2sockaddr(const char *host, int port, int ai_flags)
r = xmalloc(offsetof(len_and_sockaddr, sa) + result->ai_addrlen);
r->len = result->ai_addrlen;
memcpy(&r->sa, result->ai_addr, result->ai_addrlen);
- set_port(r, htons(port));
+ set_nport(r, htons(port));
freeaddrinfo(result);
return r;
}
@@ -161,19 +175,27 @@ static len_and_sockaddr* dotted2sockaddr(const char *host, int port)
return str2sockaddr(host, port, NI_NUMERICHOST);
}
-static int xsocket_stream(len_and_sockaddr *lsa)
+int xsocket_stream(len_and_sockaddr **lsap)
{
+ len_and_sockaddr *lsa;
int fd;
+ int len = sizeof(struct sockaddr_in);
+ int family = AF_INET;
+
#if ENABLE_FEATURE_IPV6
fd = socket(AF_INET6, SOCK_STREAM, 0);
- lsa->sa.sa_family = AF_INET6;
- lsa->len = sizeof(struct sockaddr_in6);
- if (fd >= 0)
- return fd;
+ if (fd >= 0) {
+ len = sizeof(struct sockaddr_in6);
+ family = AF_INET6;
+ } else
#endif
- fd = xsocket(AF_INET, SOCK_STREAM, 0);
- lsa->sa.sa_family = AF_INET;
- lsa->len = sizeof(struct sockaddr_in);
+ {
+ fd = xsocket(AF_INET, SOCK_STREAM, 0);
+ }
+ lsa = xzalloc(offsetof(len_and_sockaddr, sa) + len);
+ lsa->len = len;
+ lsa->sa.sa_family = family;
+ *lsap = lsa;
return fd;
}
@@ -190,11 +212,7 @@ int create_and_bind_stream_or_die(const char *bindaddr, int port)
/* user specified bind addr dictates family */
fd = xsocket(lsa->sa.sa_family, SOCK_STREAM, 0);
} else {
- lsa = xzalloc(offsetof(len_and_sockaddr, sa) +
- USE_FEATURE_IPV6(sizeof(struct sockaddr_in6))
- SKIP_FEATURE_IPV6(sizeof(struct sockaddr_in))
- );
- fd = xsocket_stream(lsa);
+ fd = xsocket_stream(&lsa);
}
setsockopt_reuseaddr(fd);
xbind(fd, &lsa->sa, lsa->len);
diff --git a/networking/ftpgetput.c b/networking/ftpgetput.c
index 51e230296..0e2d4173f 100644
--- a/networking/ftpgetput.c
+++ b/networking/ftpgetput.c
@@ -88,7 +88,7 @@ static int xconnect_ftpdata(ftp_host_info_t *server, char *buf)
*buf_ptr = '\0';
port_num += xatoul_range(buf_ptr + 1, 0, 255) * 256;
- set_port(server->lsa, htons(port_num));
+ set_nport(server->lsa, htons(port_num));
return xconnect_stream(server->lsa);
}
diff --git a/networking/nc.c b/networking/nc.c
index e1c22839c..37e658d4b 100644
--- a/networking/nc.c
+++ b/networking/nc.c
@@ -9,6 +9,10 @@
#include "busybox.h"
+/* Lots of small differences in features
+ * when compared to "standard" nc
+ */
+
static void timeout(int signum)
{
bb_error_msg_and_die("timed out");
@@ -16,7 +20,8 @@ static void timeout(int signum)
int nc_main(int argc, char **argv)
{
- int sfd = 0;
+ /* sfd sits _here_ only because of "repeat" option (-l -l). */
+ int sfd = sfd; /* for gcc */
int cfd = 0;
SKIP_NC_SERVER(const) unsigned do_listen = 0;
SKIP_NC_SERVER(const) unsigned lport = 0;
@@ -24,12 +29,10 @@ int nc_main(int argc, char **argv)
SKIP_NC_EXTRA (const) unsigned delay = 0;
SKIP_NC_EXTRA (const int execparam = 0;)
USE_NC_EXTRA (char **execparam = NULL;)
- struct sockaddr_in address;
+ len_and_sockaddr *lsa;
fd_set readfds, testfds;
int opt; /* must be signed (getopt returns -1) */
- memset(&address, 0, sizeof(address));
-
if (ENABLE_NC_SERVER || ENABLE_NC_EXTRA) {
/* getopt32 is _almost_ usable:
** it cannot handle "... -e prog -prog-opt" */
@@ -39,7 +42,6 @@ int nc_main(int argc, char **argv)
if (ENABLE_NC_SERVER && opt=='l') USE_NC_SERVER(do_listen++);
else if (ENABLE_NC_SERVER && opt=='p') {
USE_NC_SERVER(lport = bb_lookup_port(optarg, "tcp", 0));
- USE_NC_SERVER(lport = htons(lport));
}
else if (ENABLE_NC_EXTRA && opt=='w') USE_NC_EXTRA( wsecs = xatou(optarg));
else if (ENABLE_NC_EXTRA && opt=='i') USE_NC_EXTRA( delay = xatou(optarg));
@@ -71,9 +73,11 @@ int nc_main(int argc, char **argv)
// -l and -f don't mix
if (do_listen && cfd) bb_show_usage();
// Listen or file modes need zero arguments, client mode needs 2
- opt = ((do_listen || cfd) ? 0 : 2);
- if (argc != opt)
- bb_show_usage();
+ if (do_listen || cfd) {
+ if (argc) bb_show_usage();
+ } else {
+ if (!argc || argc > 2) bb_show_usage();
+ }
} else {
if (argc != 3) bb_show_usage();
argc--;
@@ -86,46 +90,37 @@ int nc_main(int argc, char **argv)
}
if (!cfd) {
- sfd = xsocket(AF_INET, SOCK_STREAM, 0);
- fcntl(sfd, F_SETFD, FD_CLOEXEC);
- setsockopt_reuseaddr(sfd);
- address.sin_family = AF_INET;
-
- // Set local port.
-
- if (lport != 0) {
- address.sin_port = lport;
- xbind(sfd, (struct sockaddr *) &address, sizeof(address));
- }
-
if (do_listen) {
- socklen_t addrlen = sizeof(address);
-
- xlisten(sfd, do_listen);
-
- // If we didn't specify a port number, query and print it to stderr.
-
+ socklen_t addrlen;
+
+ /* create_and_bind_stream_or_die(NULL, lport)
+ * would've work wonderfully, but we need
+ * to know lsa */
+ sfd = xsocket_stream(&lsa);
+ if (lport)
+ set_nport(lsa, htons(lport));
+ setsockopt_reuseaddr(sfd);
+ xbind(sfd, &lsa->sa, lsa->len);
+ xlisten(sfd, do_listen); /* can be > 1 */
+ /* If we didn't specify a port number,
+ * query and print it after listen() */
if (!lport) {
- socklen_t len = sizeof(address);
- getsockname(sfd, (struct sockaddr *) &address, &len);
- fdprintf(2, "%d\n", SWAP_BE16(address.sin_port));
+ addrlen = lsa->len;
+ getsockname(sfd, &lsa->sa, &addrlen);
+ lport = get_nport(lsa);
+ fdprintf(2, "%d\n", ntohs(lport));
}
- repeatyness:
- cfd = accept(sfd, (struct sockaddr *) &address, &addrlen);
+ fcntl(sfd, F_SETFD, FD_CLOEXEC);
+ accept_again:
+ addrlen = lsa->len;
+ cfd = accept(sfd, NULL, 0); /* &lsa->sa, &addrlen); */
if (cfd < 0)
bb_perror_msg_and_die("accept");
-
- if (!execparam) close(sfd);
+ if (!execparam)
+ close(sfd);
} else {
- struct hostent *hostinfo;
- hostinfo = xgethostbyname(argv[0]);
-
- address.sin_addr = *(struct in_addr *) *hostinfo->h_addr_list;
- address.sin_port = bb_lookup_port(argv[1], "tcp", 0);
- address.sin_port = htons(address.sin_port);
-
- xconnect(sfd, (struct sockaddr *) &address, sizeof(address));
- cfd = sfd;
+ cfd = create_and_connect_stream_or_die(argv[0],
+ argv[1] ? bb_lookup_port(argv[1], "tcp", 0) : 0);
}
}
@@ -136,17 +131,10 @@ int nc_main(int argc, char **argv)
/* -e given? */
if (execparam) {
- if (cfd) {
- signal(SIGCHLD, SIG_IGN);
- dup2(cfd, 0);
- close(cfd);
- }
- dup2(0, 1);
- dup2(0, 2);
-
+ signal(SIGCHLD, SIG_IGN);
// With more than one -l, repeatedly act as server.
-
if (do_listen > 1 && vfork()) {
+ /* parent */
// This is a bit weird as cleanup goes, since we wind up with no
// stdin/stdout/stderr. But it's small and shouldn't hurt anything.
// We check for cfd == 0 above.
@@ -154,9 +142,15 @@ int nc_main(int argc, char **argv)
close(0);
close(1);
close(2);
-
- goto repeatyness;
+ goto accept_again;
}
+ /* child (or main thread if no multiple -l) */
+ if (cfd) {
+ dup2(cfd, 0);
+ close(cfd);
+ }
+ dup2(0, 1);
+ dup2(0, 2);
USE_NC_EXTRA(execvp(execparam[0], execparam);)
/* Don't print stuff or it will go over the wire.... */
_exit(127);
@@ -184,7 +178,8 @@ int nc_main(int argc, char **argv)
sizeof(bb_common_bufsiz1));
if (fd == cfd) {
- if (nread<1) exit(0);
+ if (nread < 1)
+ exit(0);
ofd = STDOUT_FILENO;
} else {
if (nread<1) {
diff --git a/networking/wget.c b/networking/wget.c
index 22e610699..cc768db8b 100644
--- a/networking/wget.c
+++ b/networking/wget.c
@@ -432,7 +432,7 @@ int wget_main(int argc, char **argv)
s = strrchr(buf, ',');
if (!s) goto pasv_error;
port += xatou_range(s+1, 0, 255) * 256;
- set_port(lsa, htons(port));
+ set_nport(lsa, htons(port));
dfp = open_socket(lsa);
if (beg_range) {