From 5de9e9ce0bcc5aacb53ec45e36b3fa6b5c7ab5f6 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Mon, 22 Jan 2007 22:46:04 +0000 Subject: [PATCH] clean up commented out old code --- include/libbb.h | 6 +- networking/Config.in | 8 + networking/Kbuild | 1 - networking/fakeidentd.c | 378 +----------------------------------------------- networking/nslookup.c | 55 +------ networking/ping6.c | 19 +-- scripts/defconfig | 1 + 7 files changed, 16 insertions(+), 452 deletions(-) diff --git a/include/libbb.h b/include/libbb.h index 1a5ce24..8cd4bc3 100644 --- a/include/libbb.h +++ b/include/libbb.h @@ -336,12 +336,10 @@ char* xmalloc_sockaddr2hostonly_noport(const struct sockaddr *sa, socklen_t sale char* xmalloc_sockaddr2dotted(const struct sockaddr *sa, socklen_t salen); char* xmalloc_sockaddr2dotted_noport(const struct sockaddr *sa, socklen_t salen); // "old" (ipv4 only) API -//void bb_lookup_host(struct sockaddr_in *s_in, const char *host); -//int xconnect_tcp_v4(struct sockaddr_in *s_addr); // users: traceroute.c hostname.c ifconfig.c ping.c struct hostent *xgethostbyname(const char *name); -//// ping6 is the only user - convert to new API -//struct hostent *xgethostbyname2(const char *name, int af); +//TODO: eliminate gethostbyname2 in arping (the only remaining place), +//use host_and_af2sockaddr instead. extern char *xstrdup(const char *s); diff --git a/networking/Config.in b/networking/Config.in index 3b00f03..6bb2d3b 100644 --- a/networking/Config.in +++ b/networking/Config.in @@ -12,6 +12,14 @@ config FEATURE_IPV6 Enable IPv6 support in busybox. This adds IPv6 support in the networking applets. +config VERBOSE_RESOLUTION_ERRORS + bool "Verbose resolution errors" + default n + help + Enable if you are not satisfied with simplistic + "can't resolve 'hostname.com'" and want to know more. + This may increase size of your executable a bit. + config ARP bool "arp" default n diff --git a/networking/Kbuild b/networking/Kbuild index bb024c9..65ecfbf 100644 --- a/networking/Kbuild +++ b/networking/Kbuild @@ -9,7 +9,6 @@ lib-$(CONFIG_ARP) += arp.o interface.o lib-$(CONFIG_ARPING) += arping.o lib-$(CONFIG_DNSD) += dnsd.o lib-$(CONFIG_ETHER_WAKE) += ether-wake.o -#lib-$(CONFIG_FAKEIDENTD) += fakeidentd.o lib-$(CONFIG_FAKEIDENTD) += isrv_identd.o isrv.o lib-$(CONFIG_FTPGET) += ftpgetput.o lib-$(CONFIG_FTPPUT) += ftpgetput.o diff --git a/networking/fakeidentd.c b/networking/fakeidentd.c index 6f766a8..7af2f75 100644 --- a/networking/fakeidentd.c +++ b/networking/fakeidentd.c @@ -1,377 +1 @@ -/* NB: this file is to be removed soon. See isrv_identd.c */ - -/* vi: set sw=4 ts=4: */ -/* - * A fake identd server - * - * Adapted to busybox by Thomas Lundquist - * Original Author: Tomi Ollila - * http://www.guru-group.fi/~too/sw/ - * - * Licensed under GPLv2 or later, see file LICENSE in this tarball for details. - */ - -/* Ident crash course - * - * Incoming requests are of form "6191, 23\r\n" - peer asks us - * "which user connected from your port 6191 to my port 23?" - * We should answer: - * "6193, 23 : USERID : UNIX : username\r\n" - * and close the connection. - * We can also reply: - * "6195, 23 : USERID : OTHER[,US-ASCII] : username\r\n" - * "6195, 23 : ERROR : INVALID-PORT/NO-USER/HIDDEN-USER/UNKNOWN-ERROR\r\n" - * but we probably will never want that. - */ - -#include "busybox.h" - -#define SANE_INETD_ONLY_VERSION - -#ifdef SANE_INETD_ONLY_VERSION - -int fakeidentd_main(int argc, char **argv) -{ - char buf[64]; - const char *bogouser = "nobody"; - char *cur = buf; - int rem = sizeof(buf)-1; - - if (argv[1]) - bogouser = argv[1]; - - alarm(30); - while (1) { - char *p; - int sz = safe_read(0, cur, rem); - if (sz < 0) return 1; - cur[sz] = '\0'; - p = strpbrk(cur, "\r\n"); - if (p) { - *p = '\0'; - break; - } - cur += sz; - rem -= sz; - if (!rem || !sz) - break; - } - printf("%s : USERID : UNIX : %s\r\n", buf, bogouser); - return 0; -} - -#else - -/* Welcome to the bloaty horrors */ - -#include -#include - -#define MAXCONNS 20 -#define MAXIDLETIME 45 - -static const char ident_substr[] = " : USERID : UNIX : "; -enum { ident_substr_len = sizeof(ident_substr) - 1 }; -#define PIDFILE "/var/run/identd.pid" - -/* - * We have to track the 'first connection socket' so that we - * don't go around closing file descriptors for non-clients. - * - * descriptor setup normally - * 0 = server socket - * 1 = syslog fd (hopefully -- otherwise this won't work) - * 2 = connection socket after detached from tty. standard error before that - * 3 - 2 + MAXCONNS = rest connection sockets - * - * To try to make sure that syslog fd is what is "requested", the that fd - * is closed before openlog() call. It can only severely fail if fd 0 - * is initially closed. - */ -#define FCS 2 - -/* - * FD of the connection is always the index of the connection structure - * in `conns' array + FCS - */ -static struct { - time_t lasttime; - int len; - char buf[20]; -} conns[MAXCONNS]; - -/* When using global variables, bind those at least to a structure. */ -static struct { - const char *identuser; - fd_set readfds; - int conncnt; -} G; - -static char *bind_ip_address; - -static int chmatch(char c, char *chars) -{ - for (; *chars; chars++) - if (c == *chars) - return 1; - return 0; -} - -static int skipchars(char **p, char *chars) -{ - while (chmatch(**p, chars)) - (*p)++; - if (**p == '\r' || **p == '\n') - return 0; - return 1; -} - -static int parseAddrs(char *ptr, char **myaddr, char **heraddr) -{ - /* parse , */ - - if (!skipchars(&ptr, " \t")) - return -1; - - *myaddr = ptr; - - if (!skipchars(&ptr, "1234567890")) - return -1; - - if (!chmatch(*ptr, " \t,")) - return -1; - - *ptr++ = '\0'; - - if (!skipchars(&ptr, " \t,") ) - return -1; - - *heraddr = ptr; - - skipchars(&ptr, "1234567890"); - - if (!chmatch(*ptr, " \n\r")) - return -1; - - *ptr = '\0'; - - return 0; -} - -static void replyError(int s, char *buf) -{ - struct iovec iv[3]; - iv[0].iov_base = "0, 0 : ERROR : "; iv[0].iov_len = 15; - iv[1].iov_base = buf; iv[1].iov_len = strlen(buf); - iv[2].iov_base = "\r\n"; iv[2].iov_len = 2; - writev(s, iv, 3); -} - -static void reply(int s, char *buf) -{ - char *myaddr, *heraddr; - - myaddr = heraddr = NULL; - - if (parseAddrs(buf, &myaddr, &heraddr)) - replyError(s, "X-INVALID-REQUEST"); - else { - struct iovec iv[6]; - iv[0].iov_base = myaddr; iv[0].iov_len = strlen(myaddr); - iv[1].iov_base = ", "; iv[1].iov_len = 2; - iv[2].iov_base = heraddr; iv[2].iov_len = strlen(heraddr); - iv[3].iov_base = (void *)ident_substr; iv[3].iov_len = ident_substr_len; - iv[4].iov_base = (void *)G.identuser; iv[4].iov_len = strlen(G.identuser); - iv[5].iov_base = "\r\n"; iv[5].iov_len = 2; - writev(s, iv, 6); - } -} - -static void movefd(int from, int to) -{ - if (from != to) { - dup2(from, to); - close(from); - } -} - -static void deleteConn(int s) -{ - int i = s - FCS; - - close(s); - - G.conncnt--; - - /* - * Most of the time there is 0 connections. Most often that there - * is connections, there is just one connection. When this one connection - * closes, i == G.conncnt = 0 -> no copying. - * When there is more than one connection, the oldest connections closes - * earlier on average. When this happens, the code below starts copying - * the connection structure w/ highest index to the place which which is - * just deleted. This means that the connection structures are no longer - * in chronological order. I'd quess this means that when there is more - * than 1 connection, on average every other connection structure needs - * to be copied over the time all these connections are deleted. - */ - if (i != G.conncnt) { - memcpy(&conns[i], &conns[G.conncnt], sizeof(conns[0])); - movefd(G.conncnt + FCS, s); - } - - FD_CLR(G.conncnt + FCS, &G.readfds); -} - -static int closeOldest(void) -{ - time_t min = conns[0].lasttime; - int idx = 0; - int i; - - for (i = 1; i < MAXCONNS; i++) - if (conns[i].lasttime < min) - idx = i; - - replyError(idx + FCS, "X-SERVER-TOO-BUSY"); - close(idx + FCS); - - return idx; -} - -static int checkInput(char *buf, int len, int l) -{ - int i; - for (i = len; i < len + l; ++i) - if (buf[i] == '\n') - return 1; - return 0; -} - -/* May succeed. If not, won't care. */ -static const char *to_unlink; -static void writepid(void) -{ - int fd = open(PIDFILE, O_WRONLY|O_CREAT|O_TRUNC, 0664); - if (fd < 0) - return; - to_unlink = PIDFILE; - fdprintf(fd, "%d\n", getpid()); - close(fd); -} - -static void handlexitsigs(int signum) -{ - if (to_unlink) - if (unlink(to_unlink) < 0) - close(open(to_unlink, O_WRONLY|O_CREAT|O_TRUNC, 0644)); - exit(0); -} - -int fakeidentd_main(int argc, char **argv) -{ - int fd; - pid_t pid; - - /* FD_ZERO(&G.readfds); - in bss, already zeroed */ - FD_SET(0, &G.readfds); - - /* handle -b parameter */ - getopt32(argc, argv, "b:", &bind_ip_address); - /* handle optional REPLY STRING */ - if (optind < argc) - G.identuser = argv[optind]; - else - G.identuser = "nobody"; - - writepid(); - signal(SIGTERM, handlexitsigs); - signal(SIGINT, handlexitsigs); - signal(SIGQUIT, handlexitsigs); - signal(SIGHUP, SIG_IGN); - signal(SIGPIPE, SIG_IGN); /* ignore closed connections when writing */ - - fd = create_and_bind_stream_or_die(bind_ip_address, bb_lookup_port("identd", "tcp", 113)); - xlisten(fd, 5); - - pid = fork(); - if (pid < 0) - bb_perror_msg_and_die("fork"); - if (pid != 0) /* parent */ - exit(0); - /* child */ - setsid(); - movefd(fd, 0); - while (fd) - close(fd--); - openlog(applet_name, 0, LOG_DAEMON); - logmode = LOGMODE_SYSLOG; - - /* main loop where we process all events and never exit */ - while (1) { - fd_set rfds = G.readfds; - struct timeval tv = { 15, 0 }; - int i; - int tim = time(NULL); - - select(G.conncnt + FCS, &rfds, NULL, NULL, G.conncnt? &tv: NULL); - - for (i = G.conncnt - 1; i >= 0; i--) { - int s = i + FCS; - - if (FD_ISSET(s, &rfds)) { - char *buf = conns[i].buf; - unsigned len = conns[i].len; - unsigned l; - - l = read(s, buf + len, sizeof(conns[0].buf) - len); - if (l > 0) { - if (checkInput(buf, len, l)) { - reply(s, buf); - goto deleteconn; - } else if (len + l >= sizeof(conns[0].buf)) { - replyError(s, "X-INVALID-REQUEST"); - goto deleteconn; - } else { - conns[i].len += l; - } - } else { - goto deleteconn; - } - conns[i].lasttime = tim; - continue; -deleteconn: - deleteConn(s); - } else { - /* implement as time_after() in linux kernel sources ... */ - if (conns[i].lasttime + MAXIDLETIME <= tim) { - replyError(s, "X-TIMEOUT"); - deleteConn(s); - } - } - } - - if (FD_ISSET(0, &rfds)) { - int s = accept(0, NULL, 0); - - if (s < 0) { - if (errno != EINTR) - bb_perror_msg("accept"); - } else { - if (G.conncnt == MAXCONNS) - i = closeOldest(); - else - i = G.conncnt++; - - movefd(s, i + FCS); /* move if not already there */ - FD_SET(i + FCS, &G.readfds); - conns[i].len = 0; - conns[i].lasttime = time(NULL); - } - } - } /* end of while (1) */ - - return 0; -} - -#endif /* !SANE_INETD_ONLY_VERSION */ +/* TO DELETE */ diff --git a/networking/nslookup.c b/networking/nslookup.c index af08162..914f3f5 100644 --- a/networking/nslookup.c +++ b/networking/nslookup.c @@ -47,57 +47,8 @@ * ns3.kernel.org internet address = 204.152.191.36 */ -/*static int sockaddr_to_dotted(struct sockaddr *saddr, char *buf, int buflen) -{ - if (buflen <= 0) return -1; - buf[0] = '\0'; - if (saddr->sa_family == AF_INET) { - inet_ntop(AF_INET, &((struct sockaddr_in*)saddr)->sin_addr, buf, buflen); - return 0; - } - if (saddr->sa_family == AF_INET6) { - inet_ntop(AF_INET6, &((struct sockaddr_in6*)saddr)->sin6_addr, buf, buflen); - return 0; - } - return -1; -} -*/ - static int print_host(const char *hostname, const char *header) { -#if 0 - char str[128]; /* IPv6 address will fit, hostnames hopefully too */ - struct addrinfo *result = NULL; - int rc; - struct addrinfo hint; - - memset(&hint, 0 , sizeof(hint)); - /* hint.ai_family = AF_UNSPEC; - zero anyway */ - /* Needed. Or else we will get each address thrice (or more) - * for each possible socket type (tcp,udp,raw...): */ - hint.ai_socktype = SOCK_STREAM; - // hint.ai_flags = AI_CANONNAME; - rc = getaddrinfo(hostname, NULL /*service*/, &hint, &result); - - if (!rc) { - struct addrinfo *cur = result; - // printf("%s\n", cur->ai_canonname); ? - while (cur) { - sockaddr_to_dotted(cur->ai_addr, str, sizeof(str)); - printf("%s %s\nAddress: %s", header, hostname, str); - str[0] = ' '; - if (getnameinfo(cur->ai_addr, cur->ai_addrlen, str+1, sizeof(str)-1, NULL, 0, NI_NAMEREQD)) - str[0] = '\0'; - puts(str); - cur = cur->ai_next; - } - } else { - bb_error_msg("getaddrinfo('%s') failed: %s", hostname, gai_strerror(rc)); - } - freeaddrinfo(result); - return (rc != 0); - -#else /* We can't use host2sockaddr() - we want to get ALL addresses, * not just one */ @@ -136,7 +87,7 @@ static int print_host(const char *hostname, const char *header) } } else { #if ENABLE_VERBOSE_RESOLUTION_ERRORS - bb_error_msg("getaddrinfo('%s') failed: %s", hostname, gai_strerror(rc)); + bb_error_msg("can't resolve '%s': %s", hostname, gai_strerror(rc)); #else bb_error_msg("can't resolve '%s'", hostname); #endif @@ -144,10 +95,8 @@ static int print_host(const char *hostname, const char *header) if (ENABLE_FEATURE_CLEAN_UP) freeaddrinfo(result); return (rc != 0); -#endif } - /* lookup the default nameserver and display it */ static void server_print(void) { @@ -167,7 +116,6 @@ static void server_print(void) puts(""); } - /* alter the global _res nameserver structure to use an explicit dns server instead of what is in /etc/resolv.h */ static void set_default_dns(char *server) @@ -180,7 +128,6 @@ static void set_default_dns(char *server) } } - int nslookup_main(int argc, char **argv) { /* We allow 1 or 2 arguments. diff --git a/networking/ping6.c b/networking/ping6.c index ccb19b2..b95e098 100644 --- a/networking/ping6.c +++ b/networking/ping6.c @@ -48,8 +48,6 @@ static void ping(const char *host); /* simple version */ -//static struct hostent *h; - static void noresp(int ign) { printf("No response from %s\n", h->h_name); @@ -58,7 +56,7 @@ static void noresp(int ign) static void ping(const char *host) { - len_and_sockaddr *lsa;// + len_and_sockaddr *lsa; struct sockaddr_in6 pingaddr; struct icmp6_hdr *pkt; int pingsock, c; @@ -67,10 +65,6 @@ static void ping(const char *host) pingsock = create_icmp6_socket(); - //memset(&pingaddr, 0, sizeof(pingaddr)); - //pingaddr.sin6_family = AF_INET6; - //h = xgethostbyname2(host, AF_INET6); - //memcpy(&pingaddr.sin6_addr, h->h_addr, sizeof(pingaddr.sin6_addr)); lsa = host_and_af2sockaddr(host, 0, AF_INET6); pingaddr = lsa->sin6; @@ -144,8 +138,7 @@ static int myid; static unsigned long tmin = ULONG_MAX, tmax, tsum; static char rcvd_tbl[MAX_DUP_CHK / 8]; -//static struct hostent *hostent; -char *hostname; +static char *hostname; static void sendping(int); static void pingstats(int); @@ -318,7 +311,7 @@ static void unpack(char *packet, int sz, struct sockaddr_in6 *from, int hoplimit extern int BUG_bad_offsetof_icmp6_cksum(void); static void ping(const char *host) { - len_and_sockaddr *lsa;// + len_and_sockaddr *lsa; char packet[datalen + MAXIPLEN + MAXICMPLEN]; char buf[INET6_ADDRSTRLEN]; int sockopt; @@ -329,12 +322,6 @@ static void ping(const char *host) pingsock = create_icmp6_socket(); - //memset(&pingaddr, 0, sizeof(pingaddr)); - //pingaddr.sin6_family = AF_INET6; - //hostent = xgethostbyname2(host, AF_INET6); - //if (hostent->h_addrtype != AF_INET6) - // bb_error_msg_and_die("unknown address type; only AF_INET6 is currently supported"); - //memcpy(&pingaddr.sin6_addr, hostent->h_addr, sizeof(pingaddr.sin6_addr)); lsa = host_and_af2sockaddr(host, 0, AF_INET6); hostname = xmalloc_sockaddr2host_noport(&lsa->sa, lsa->len); pingaddr = lsa->sin6; diff --git a/scripts/defconfig b/scripts/defconfig index 8f15481..6a1c57b 100644 --- a/scripts/defconfig +++ b/scripts/defconfig @@ -501,6 +501,7 @@ CONFIG_WATCHDOG=y # Networking Utilities # CONFIG_FEATURE_IPV6=y +# CONFIG_VERBOSE_RESOLUTION_ERRORS is not set CONFIG_ARP=y CONFIG_ARPING=y CONFIG_DNSD=y -- 2.7.4