From e491611316943fb5a44c94f27efecb9a9117a3c5 Mon Sep 17 00:00:00 2001 From: Peter Korsgaard Date: Thu, 11 Jun 2009 12:34:01 +0200 Subject: [PATCH] busybox: additional 1.14.1 fixes Signed-off-by: Peter Korsgaard --- package/busybox/busybox-1.14.1-httpd.patch | 733 +++++++++++++++++++ package/busybox/busybox-1.14.1-telnetd.patch | 85 +++ 2 files changed, 818 insertions(+) create mode 100644 package/busybox/busybox-1.14.1-httpd.patch create mode 100644 package/busybox/busybox-1.14.1-telnetd.patch diff --git a/package/busybox/busybox-1.14.1-httpd.patch b/package/busybox/busybox-1.14.1-httpd.patch new file mode 100644 index 0000000000..ff53f0d839 --- /dev/null +++ b/package/busybox/busybox-1.14.1-httpd.patch @@ -0,0 +1,733 @@ +diff -urpN busybox-1.14.1/networking/httpd.c busybox-1.14.1-httpd/networking/httpd.c +--- busybox-1.14.1/networking/httpd.c 2009-05-27 18:00:23.000000000 +0200 ++++ busybox-1.14.1-httpd/networking/httpd.c 2009-06-09 20:40:57.000000000 +0200 +@@ -32,7 +32,7 @@ + * foo=`httpd -d $foo` # decode "Hello%20World" as "Hello World" + * bar=`httpd -e ""` # encode as "<Hello World>" + * Note that url encoding for arguments is not the same as html encoding for +- * presentation. -d decodes a url-encoded argument while -e encodes in html ++ * presentation. -d decodes an url-encoded argument while -e encodes in html + * for page display. + * + * httpd.conf has the following format: +@@ -54,7 +54,7 @@ + * /adm:admin:setup # Require user admin, pwd setup on urls starting with /adm/ + * /adm:toor:PaSsWd # or user toor, pwd PaSsWd on urls starting with /adm/ + * .au:audio/basic # additional mime type for audio.au files +- * *.php:/path/php # running cgi.php scripts through an interpreter ++ * *.php:/path/php # run xxx.php through an interpreter + * + * A/D may be as a/d or allow/deny - only first char matters. + * Deny/Allow IP logic: +@@ -94,13 +94,13 @@ + * server exits with an error. + * + */ ++ /* TODO: use TCP_CORK, parse_config() */ + + #include "libbb.h" + #if ENABLE_FEATURE_HTTPD_USE_SENDFILE + # include + #endif + +-//#define DEBUG 1 + #define DEBUG 0 + + #define IOBUF_SIZE 8192 /* IO buffer */ +@@ -115,8 +115,8 @@ + + #define HEADER_READ_TIMEOUT 60 + +-static const char default_path_httpd_conf[] ALIGN1 = "/etc"; +-static const char httpd_conf[] ALIGN1 = "httpd.conf"; ++static const char DEFAULT_PATH_HTTPD_CONF[] ALIGN1 = "/etc"; ++static const char HTTPD_CONF[] ALIGN1 = "httpd.conf"; + static const char HTTP_200[] ALIGN1 = "HTTP/1.0 200 OK\r\n"; + + typedef struct has_next_ptr { +@@ -242,7 +242,7 @@ struct globals { + const char *bind_addr_or_port; + + const char *g_query; +- const char *configFile; ++ const char *opt_c_configFile; + const char *home_httpd; + const char *index_page; + +@@ -250,13 +250,13 @@ struct globals { + const char *found_moved_temporarily; + Htaccess_IP *ip_a_d; /* config allow/deny lines */ + +- USE_FEATURE_HTTPD_BASIC_AUTH(const char *g_realm;) +- USE_FEATURE_HTTPD_BASIC_AUTH(char *remoteuser;) +- USE_FEATURE_HTTPD_CGI(char *referer;) +- USE_FEATURE_HTTPD_CGI(char *user_agent;) +- USE_FEATURE_HTTPD_CGI(char *host;) +- USE_FEATURE_HTTPD_CGI(char *http_accept;) +- USE_FEATURE_HTTPD_CGI(char *http_accept_language;) ++ IF_FEATURE_HTTPD_BASIC_AUTH(const char *g_realm;) ++ IF_FEATURE_HTTPD_BASIC_AUTH(char *remoteuser;) ++ IF_FEATURE_HTTPD_CGI(char *referer;) ++ IF_FEATURE_HTTPD_CGI(char *user_agent;) ++ IF_FEATURE_HTTPD_CGI(char *host;) ++ IF_FEATURE_HTTPD_CGI(char *http_accept;) ++ IF_FEATURE_HTTPD_CGI(char *http_accept_language;) + + off_t file_size; /* -1 - unknown */ + #if ENABLE_FEATURE_HTTPD_RANGES +@@ -289,7 +289,7 @@ struct globals { + #define rmt_ip (G.rmt_ip ) + #define bind_addr_or_port (G.bind_addr_or_port) + #define g_query (G.g_query ) +-#define configFile (G.configFile ) ++#define opt_c_configFile (G.opt_c_configFile ) + #define home_httpd (G.home_httpd ) + #define index_page (G.index_page ) + #define found_mime_type (G.found_mime_type ) +@@ -326,7 +326,7 @@ enum { + #define proxy (G.proxy ) + #define INIT_G() do { \ + SET_PTR_TO_GLOBALS(xzalloc(sizeof(G))); \ +- USE_FEATURE_HTTPD_BASIC_AUTH(g_realm = "Web Server Authentication";) \ ++ IF_FEATURE_HTTPD_BASIC_AUTH(g_realm = "Web Server Authentication";) \ + bind_addr_or_port = "80"; \ + index_page = "index.html"; \ + file_size = -1; \ +@@ -452,14 +452,6 @@ static int scan_ip_mask(const char *str, + /* + * Parse configuration file into in-memory linked list. + * +- * The first non-white character is examined to determine if the config line +- * is one of the following: +- * .ext:mime/type # new mime type not compiled into httpd +- * [adAD]:from # ip address allow/deny, * for wildcard +- * /path:user:pass # username/password +- * Ennn:error.html # error page for status nnn +- * P:/url:[http://]hostname[:port]/new/path # reverse proxy +- * + * Any previous IP rules are discarded. + * If the flag argument is not SUBDIR_PARSE then all /path and mime rules + * are also discarded. That is, previous settings are retained if flag is +@@ -469,99 +461,150 @@ static int scan_ip_mask(const char *str, + * path Path where to look for httpd.conf (without filename). + * flag Type of the parse request. + */ +-/* flag */ +-#define FIRST_PARSE 0 +-#define SUBDIR_PARSE 1 +-#define SIGNALED_PARSE 2 +-#define FIND_FROM_HTTPD_ROOT 3 ++/* flag param: */ ++enum { ++ FIRST_PARSE = 0, /* path will be "/etc" */ ++ SIGNALED_PARSE = 1, /* path will be "/etc" */ ++ SUBDIR_PARSE = 2, /* path will be derived from URL */ ++}; + static void parse_conf(const char *path, int flag) + { ++ /* internally used extra flag state */ ++ enum { TRY_CURDIR_PARSE = 3 }; ++ + FILE *f; +-#if ENABLE_FEATURE_HTTPD_BASIC_AUTH +- Htaccess *prev; +-#endif +- Htaccess *cur; +- const char *filename = configFile; ++ const char *filename; + char buf[160]; +- char *p, *p0; +- char *after_colon; +- Htaccess_IP *pip; + + /* discard old rules */ + free_Htaccess_IP_list(&ip_a_d); + flg_deny_all = 0; + /* retain previous auth and mime config only for subdir parse */ + if (flag != SUBDIR_PARSE) { ++ free_Htaccess_list(&mime_a); + #if ENABLE_FEATURE_HTTPD_BASIC_AUTH + free_Htaccess_list(&g_auth); + #endif +- free_Htaccess_list(&mime_a); + #if ENABLE_FEATURE_HTTPD_CONFIG_WITH_SCRIPT_INTERPR + free_Htaccess_list(&script_i); + #endif + } + ++ filename = opt_c_configFile; + if (flag == SUBDIR_PARSE || filename == NULL) { +- filename = alloca(strlen(path) + sizeof(httpd_conf) + 2); +- sprintf((char *)filename, "%s/%s", path, httpd_conf); ++ filename = alloca(strlen(path) + sizeof(HTTPD_CONF) + 2); ++ sprintf((char *)filename, "%s/%s", path, HTTPD_CONF); + } + + while ((f = fopen_for_read(filename)) == NULL) { +- if (flag == SUBDIR_PARSE || flag == FIND_FROM_HTTPD_ROOT) { ++ if (flag >= SUBDIR_PARSE) { /* SUBDIR or TRY_CURDIR */ + /* config file not found, no changes to config */ + return; + } +- if (configFile && flag == FIRST_PARSE) /* if -c option given */ +- bb_simple_perror_msg_and_die(filename); +- flag = FIND_FROM_HTTPD_ROOT; +- filename = httpd_conf; ++ if (flag == FIRST_PARSE) { ++ /* -c CONFFILE given, but CONFFILE doesn't exist? */ ++ if (opt_c_configFile) ++ bb_simple_perror_msg_and_die(opt_c_configFile); ++ /* else: no -c, thus we looked at /etc/httpd.conf, ++ * and it's not there. try ./httpd.conf: */ ++ } ++ flag = TRY_CURDIR_PARSE; ++ filename = HTTPD_CONF; + } + + #if ENABLE_FEATURE_HTTPD_BASIC_AUTH +- prev = g_auth; +-#endif +- /* This could stand some work */ +- while ((p0 = fgets(buf, sizeof(buf), f)) != NULL) { +- after_colon = NULL; +- for (p = p0; *p0 != '\0' && *p0 != '#'; p0++) { +- if (!isspace(*p0)) { +- *p++ = *p0; +- if (*p0 == ':' && after_colon == NULL) +- after_colon = p; ++ /* in "/file:user:pass" lines, we prepend path in subdirs */ ++ if (flag != SUBDIR_PARSE) ++ path = ""; ++#endif ++ /* The lines can be: ++ * ++ * I:default_index_file ++ * H:http_home ++ * [AD]:IP[/mask] # allow/deny, * for wildcard ++ * Ennn:error.html # error page for status nnn ++ * P:/url:[http://]hostname[:port]/new/path # reverse proxy ++ * .ext:mime/type # mime type ++ * *.php:/path/php # run xxx.php through an interpreter ++ * /file:user:pass # username and password ++ */ ++ while (fgets(buf, sizeof(buf), f) != NULL) { ++ unsigned strlen_buf; ++ unsigned char ch; ++ char *after_colon; ++ ++ { /* remove all whitespace, and # comments */ ++ char *p, *p0; ++ ++ p0 = buf; ++ /* skip non-whitespace beginning. Often the whole line ++ * is non-whitespace. We want this case to work fast, ++ * without needless copying, therefore we don't merge ++ * this operation into next while loop. */ ++ while ((ch = *p0) != '\0' && ch != '\n' && ch != '#' ++ && ch != ' ' && ch != '\t' ++ ) { ++ p0++; ++ } ++ p = p0; ++ /* if we enter this loop, we have some whitespace. ++ * discard it */ ++ while (ch != '\0' && ch != '\n' && ch != '#') { ++ if (ch != ' ' && ch != '\t') { ++ *p++ = ch; ++ } ++ ch = *++p0; + } ++ *p = '\0'; ++ strlen_buf = p - buf; ++ if (strlen_buf == 0) ++ continue; /* empty line */ + } +- *p = '\0'; + +- /* test for empty or strange line */ +- if (after_colon == NULL || *after_colon == '\0') ++ after_colon = strchr(buf, ':'); ++ /* strange line? */ ++ if (after_colon == NULL || *++after_colon == '\0') ++ goto config_error; ++ ++ ch = (buf[0] & ~0x20); /* toupper if it's a letter */ ++ ++ if (ch == 'I') { ++ index_page = xstrdup(after_colon); + continue; +- p0 = buf; +- if (*p0 == 'd' || *p0 == 'a') +- *p0 -= 0x20; /* a/d -> A/D */ +- if (*after_colon == '*') { +- if (*p0 == 'D') { +- /* memorize "deny all" */ +- flg_deny_all = 1; +- } +- /* skip assumed "A:*", it is a default anyway */ ++ } ++ ++ /* do not allow jumping around using H in subdir's configs */ ++ if (flag == FIRST_PARSE && ch == 'H') { ++ home_httpd = xstrdup(after_colon); ++ xchdir(home_httpd); + continue; + } + +- if (*p0 == 'A' || *p0 == 'D') { +- /* storing current config IP line */ +- pip = xzalloc(sizeof(Htaccess_IP)); +- if (scan_ip_mask(after_colon, &(pip->ip), &(pip->mask))) { ++ if (ch == 'A' || ch == 'D') { ++ Htaccess_IP *pip; ++ ++ if (*after_colon == '*') { ++ if (ch == 'D') { ++ /* memorize "deny all" */ ++ flg_deny_all = 1; ++ } ++ /* skip assumed "A:*", it is a default anyway */ ++ continue; ++ } ++ /* store "allow/deny IP/mask" line */ ++ pip = xzalloc(sizeof(*pip)); ++ if (scan_ip_mask(after_colon, &pip->ip, &pip->mask)) { + /* IP{/mask} syntax error detected, protect all */ +- *p0 = 'D'; ++ ch = 'D'; + pip->mask = 0; + } +- pip->allow_deny = *p0; +- if (*p0 == 'D') { ++ pip->allow_deny = ch; ++ if (ch == 'D') { + /* Deny:from_IP - prepend */ + pip->next = ip_a_d; + ip_a_d = pip; + } else { +- /* A:from_IP - append (thus D precedes A) */ ++ /* A:from_IP - append (thus all D's precedes A's) */ + Htaccess_IP *prev_IP = ip_a_d; + if (prev_IP == NULL) { + ip_a_d = pip; +@@ -575,12 +618,12 @@ static void parse_conf(const char *path, + } + + #if ENABLE_FEATURE_HTTPD_ERROR_PAGES +- if (flag == FIRST_PARSE && *p0 == 'E') { ++ if (flag == FIRST_PARSE && ch == 'E') { + unsigned i; +- int status = atoi(++p0); /* error status code */ ++ int status = atoi(buf + 1); /* error status code */ ++ + if (status < HTTP_CONTINUE) { +- bb_error_msg("config error '%s' in '%s'", buf, filename); +- continue; ++ goto config_error; + } + /* then error page; find matching status */ + for (i = 0; i < ARRAY_SIZE(http_response_type); i++) { +@@ -597,7 +640,7 @@ static void parse_conf(const char *path, + #endif + + #if ENABLE_FEATURE_HTTPD_PROXY +- if (flag == FIRST_PARSE && *p0 == 'P') { ++ if (flag == FIRST_PARSE && ch == 'P') { + /* P:/url:[http://]hostname[:port]/new/path */ + char *url_from, *host_port, *url_to; + Htaccess_Proxy *proxy_entry; +@@ -605,23 +648,20 @@ static void parse_conf(const char *path, + url_from = after_colon; + host_port = strchr(after_colon, ':'); + if (host_port == NULL) { +- bb_error_msg("config error '%s' in '%s'", buf, filename); +- continue; ++ goto config_error; + } + *host_port++ = '\0'; + if (strncmp(host_port, "http://", 7) == 0) + host_port += 7; + if (*host_port == '\0') { +- bb_error_msg("config error '%s' in '%s'", buf, filename); +- continue; ++ goto config_error; + } + url_to = strchr(host_port, '/'); + if (url_to == NULL) { +- bb_error_msg("config error '%s' in '%s'", buf, filename); +- continue; ++ goto config_error; + } + *url_to = '\0'; +- proxy_entry = xzalloc(sizeof(Htaccess_Proxy)); ++ proxy_entry = xzalloc(sizeof(*proxy_entry)); + proxy_entry->url_from = xstrdup(url_from); + proxy_entry->host_port = xstrdup(host_port); + *url_to = '/'; +@@ -631,115 +671,87 @@ static void parse_conf(const char *path, + continue; + } + #endif ++ /* the rest of directives are non-alphabetic, ++ * must avoid using "toupper'ed" ch */ ++ ch = buf[0]; + +-#if ENABLE_FEATURE_HTTPD_BASIC_AUTH +- if (*p0 == '/') { +- /* make full path from httpd root / current_path / config_line_path */ +- const char *tp = (flag == SUBDIR_PARSE ? path : ""); +- p0 = xmalloc(strlen(tp) + (after_colon - buf) + 2 + strlen(after_colon)); +- after_colon[-1] = '\0'; +- sprintf(p0, "/%s%s", tp, buf); +- +- /* looks like bb_simplify_path... */ +- tp = p = p0; +- do { +- if (*p == '/') { +- if (*tp == '/') { /* skip duplicate (or initial) slash */ +- continue; +- } +- if (*tp == '.') { +- if (tp[1] == '/' || tp[1] == '\0') { /* remove extra '.' */ +- continue; +- } +- if ((tp[1] == '.') && (tp[2] == '/' || tp[2] == '\0')) { +- ++tp; +- if (p > p0) { +- while (*--p != '/') /* omit previous dir */ +- continue; +- } +- continue; +- } +- } +- } +- *++p = *tp; +- } while (*++tp); ++ if (ch == '.' /* ".ext:mime/type" */ ++#if ENABLE_FEATURE_HTTPD_CONFIG_WITH_SCRIPT_INTERPR ++ || (ch == '*' && buf[1] == '.') /* "*.php:/path/php" */ ++#endif ++ ) { ++ char *p; ++ Htaccess *cur; + +- if ((p == p0) || (*p != '/')) { /* not a trailing slash */ +- ++p; /* so keep last character */ ++ cur = xzalloc(sizeof(*cur) /* includes space for NUL */ + strlen_buf); ++ strcpy(cur->before_colon, buf); ++ p = cur->before_colon + (after_colon - buf); ++ p[-1] = '\0'; ++ cur->after_colon = p; ++ if (ch == '.') { ++ /* .mime line: prepend to mime_a list */ ++ cur->next = mime_a; ++ mime_a = cur; ++ } ++#if ENABLE_FEATURE_HTTPD_CONFIG_WITH_SCRIPT_INTERPR ++ else { ++ /* script interpreter line: prepend to script_i list */ ++ cur->next = script_i; ++ script_i = cur; + } +- *p = ':'; +- strcpy(p + 1, after_colon); +- } + #endif +- if (*p0 == 'I') { +- index_page = xstrdup(after_colon); +- continue; +- } +- +- /* Do not allow jumping around using H in subdir's configs */ +- if (flag == FIRST_PARSE && *p0 == 'H') { +- home_httpd = xstrdup(after_colon); +- xchdir(home_httpd); + continue; + } + +- /* storing current config line */ +- cur = xzalloc(sizeof(Htaccess) + strlen(p0)); +- strcpy(cur->before_colon, p0); +-#if ENABLE_FEATURE_HTTPD_BASIC_AUTH +- if (*p0 == '/') /* was malloced - see above */ +- free(p0); +-#endif +- cur->after_colon = strchr(cur->before_colon, ':'); +- *cur->after_colon++ = '\0'; +- if (cur->before_colon[0] == '.') { +- /* .mime line: prepend to mime_a list */ +- cur->next = mime_a; +- mime_a = cur; +- continue; +- } +-#if ENABLE_FEATURE_HTTPD_CONFIG_WITH_SCRIPT_INTERPR +- if (cur->before_colon[0] == '*' && cur->before_colon[1] == '.') { +- /* script interpreter line: prepend to script_i list */ +- cur->next = script_i; +- script_i = cur; +- continue; +- } +-#endif + #if ENABLE_FEATURE_HTTPD_BASIC_AUTH +-//TODO: we do not test for leading "/"?? +-//also, do we leak cur if BASIC_AUTH is off? +- if (prev == NULL) { +- /* first line */ +- g_auth = prev = cur; +- } else { +- /* sort path, if current length eq or bigger then move up */ +- Htaccess *prev_hti = g_auth; +- size_t l = strlen(cur->before_colon); +- Htaccess *hti; +- +- for (hti = prev_hti; hti; hti = hti->next) { +- if (l >= strlen(hti->before_colon)) { +- /* insert before hti */ +- cur->next = hti; +- if (prev_hti != hti) { +- prev_hti->next = cur; +- } else { +- /* insert as top */ +- g_auth = cur; ++ if (ch == '/') { /* "/file:user:pass" */ ++ char *p; ++ Htaccess *cur; ++ unsigned file_len; ++ ++ /* note: path is "" unless we are in SUBDIR parse, ++ * otherwise it does NOT start with "/" */ ++ cur = xzalloc(sizeof(*cur) /* includes space for NUL */ ++ + 1 + strlen(path) ++ + strlen_buf ++ ); ++ /* form "/path/file" */ ++ sprintf(cur->before_colon, "/%s%.*s", ++ path, ++ (int) (after_colon - buf - 1), /* includes "/", but not ":" */ ++ buf); ++ /* canonicalize it */ ++ p = bb_simplify_abs_path_inplace(cur->before_colon); ++ file_len = p - cur->before_colon; ++ /* add "user:pass" after NUL */ ++ strcpy(++p, after_colon); ++ cur->after_colon = p; ++ ++ /* insert cur into g_auth */ ++ /* g_auth is sorted by decreased filename length */ ++ { ++ Htaccess *auth, **authp; ++ ++ authp = &g_auth; ++ while ((auth = *authp) != NULL) { ++ if (file_len >= strlen(auth->before_colon)) { ++ /* insert cur before auth */ ++ cur->next = auth; ++ break; + } +- break; ++ authp = &auth->next; + } +- if (prev_hti != hti) +- prev_hti = prev_hti->next; +- } +- if (!hti) { /* not inserted, add to bottom */ +- prev->next = cur; +- prev = cur; ++ *authp = cur; + } ++ continue; + } + #endif /* BASIC_AUTH */ ++ ++ /* the line is not recognized */ ++ config_error: ++ bb_error_msg("config error '%s' in '%s'", buf, filename); + } /* while (fgets) */ ++ + fclose(f); + } + +@@ -1527,11 +1539,6 @@ static NOINLINE void send_file_and_exit( + send_headers_and_exit(HTTP_NOT_FOUND); + log_and_exit(); + } +- +- if (DEBUG) +- bb_error_msg("sending file '%s' content-type: %s", +- url, found_mime_type); +- + /* If you want to know about EPIPE below + * (happens if you abort downloads from local httpd): */ + signal(SIGPIPE, SIG_IGN); +@@ -1559,6 +1566,11 @@ static NOINLINE void send_file_and_exit( + } + } + } ++ ++ if (DEBUG) ++ bb_error_msg("sending file '%s' content-type: %s", ++ url, found_mime_type); ++ + #if ENABLE_FEATURE_HTTPD_RANGES + if (what == SEND_BODY) + range_start = 0; /* err pages and ranges don't mix */ +@@ -1587,14 +1599,14 @@ static NOINLINE void send_file_and_exit( + while (1) { + /* sz is rounded down to 64k */ + ssize_t sz = MAXINT(ssize_t) - 0xffff; +- USE_FEATURE_HTTPD_RANGES(if (sz > range_len) sz = range_len;) ++ IF_FEATURE_HTTPD_RANGES(if (sz > range_len) sz = range_len;) + count = sendfile(STDOUT_FILENO, fd, &offset, sz); + if (count < 0) { + if (offset == range_start) + break; /* fall back to read/write loop */ + goto fin; + } +- USE_FEATURE_HTTPD_RANGES(range_len -= sz;) ++ IF_FEATURE_HTTPD_RANGES(range_len -= sz;) + if (count == 0 || range_len == 0) + log_and_exit(); + } +@@ -1602,16 +1614,16 @@ static NOINLINE void send_file_and_exit( + #endif + while ((count = safe_read(fd, iobuf, IOBUF_SIZE)) > 0) { + ssize_t n; +- USE_FEATURE_HTTPD_RANGES(if (count > range_len) count = range_len;) ++ IF_FEATURE_HTTPD_RANGES(if (count > range_len) count = range_len;) + n = full_write(STDOUT_FILENO, iobuf, count); + if (count != n) + break; +- USE_FEATURE_HTTPD_RANGES(range_len -= count;) ++ IF_FEATURE_HTTPD_RANGES(range_len -= count;) + if (range_len == 0) + break; + } + if (count < 0) { +- USE_FEATURE_HTTPD_USE_SENDFILE(fin:) ++ IF_FEATURE_HTTPD_USE_SENDFILE(fin:) + if (verbose > 1) + bb_perror_msg("error"); + } +@@ -1839,12 +1851,12 @@ static void handle_incoming_and_exit(con + + /* Find end of URL and parse HTTP version, if any */ + http_major_version = '0'; +- USE_FEATURE_HTTPD_PROXY(http_minor_version = '0';) ++ IF_FEATURE_HTTPD_PROXY(http_minor_version = '0';) + tptr = strchrnul(urlp, ' '); + /* Is it " HTTP/"? */ + if (tptr[0] && strncmp(tptr + 1, HTTP_200, 5) == 0) { + http_major_version = tptr[6]; +- USE_FEATURE_HTTPD_PROXY(http_minor_version = tptr[8];) ++ IF_FEATURE_HTTPD_PROXY(http_minor_version = tptr[8];) + } + *tptr = '\0'; + +@@ -2031,8 +2043,8 @@ static void handle_incoming_and_exit(con + /* We are done reading headers, disable peer timeout */ + alarm(0); + +- if (strcmp(bb_basename(urlcopy), httpd_conf) == 0 || !ip_allowed) { +- /* protect listing [/path]/httpd_conf or IP deny */ ++ if (strcmp(bb_basename(urlcopy), HTTPD_CONF) == 0 || !ip_allowed) { ++ /* protect listing [/path]/httpd.conf or IP deny */ + send_headers_and_exit(HTTP_FORBIDDEN); + } + +@@ -2074,7 +2086,7 @@ static void handle_incoming_and_exit(con + header_ptr += 2; + write(proxy_fd, header_buf, header_ptr - header_buf); + free(header_buf); /* on the order of 8k, free it */ +- /* cgi_io_loop_and_exit needs to have two disctinct fds */ ++ /* cgi_io_loop_and_exit needs to have two distinct fds */ + cgi_io_loop_and_exit(proxy_fd, dup(proxy_fd), length); + } + #endif +@@ -2245,17 +2257,17 @@ static void mini_httpd_inetd(void) + + static void sighup_handler(int sig UNUSED_PARAM) + { +- parse_conf(default_path_httpd_conf, SIGNALED_PARSE); ++ parse_conf(DEFAULT_PATH_HTTPD_CONF, SIGNALED_PARSE); + } + + enum { + c_opt_config_file = 0, + d_opt_decode_url, + h_opt_home_httpd, +- USE_FEATURE_HTTPD_ENCODE_URL_STR(e_opt_encode_url,) +- USE_FEATURE_HTTPD_BASIC_AUTH( r_opt_realm ,) +- USE_FEATURE_HTTPD_AUTH_MD5( m_opt_md5 ,) +- USE_FEATURE_HTTPD_SETUID( u_opt_setuid ,) ++ IF_FEATURE_HTTPD_ENCODE_URL_STR(e_opt_encode_url,) ++ IF_FEATURE_HTTPD_BASIC_AUTH( r_opt_realm ,) ++ IF_FEATURE_HTTPD_AUTH_MD5( m_opt_md5 ,) ++ IF_FEATURE_HTTPD_SETUID( u_opt_setuid ,) + p_opt_port , + p_opt_inetd , + p_opt_foreground, +@@ -2263,10 +2275,10 @@ enum { + OPT_CONFIG_FILE = 1 << c_opt_config_file, + OPT_DECODE_URL = 1 << d_opt_decode_url, + OPT_HOME_HTTPD = 1 << h_opt_home_httpd, +- OPT_ENCODE_URL = USE_FEATURE_HTTPD_ENCODE_URL_STR((1 << e_opt_encode_url)) + 0, +- OPT_REALM = USE_FEATURE_HTTPD_BASIC_AUTH( (1 << r_opt_realm )) + 0, +- OPT_MD5 = USE_FEATURE_HTTPD_AUTH_MD5( (1 << m_opt_md5 )) + 0, +- OPT_SETUID = USE_FEATURE_HTTPD_SETUID( (1 << u_opt_setuid )) + 0, ++ OPT_ENCODE_URL = IF_FEATURE_HTTPD_ENCODE_URL_STR((1 << e_opt_encode_url)) + 0, ++ OPT_REALM = IF_FEATURE_HTTPD_BASIC_AUTH( (1 << r_opt_realm )) + 0, ++ OPT_MD5 = IF_FEATURE_HTTPD_AUTH_MD5( (1 << m_opt_md5 )) + 0, ++ OPT_SETUID = IF_FEATURE_HTTPD_SETUID( (1 << u_opt_setuid )) + 0, + OPT_PORT = 1 << p_opt_port, + OPT_INETD = 1 << p_opt_inetd, + OPT_FOREGROUND = 1 << p_opt_foreground, +@@ -2280,10 +2292,10 @@ int httpd_main(int argc UNUSED_PARAM, ch + int server_socket = server_socket; /* for gcc */ + unsigned opt; + char *url_for_decode; +- USE_FEATURE_HTTPD_ENCODE_URL_STR(const char *url_for_encode;) +- USE_FEATURE_HTTPD_SETUID(const char *s_ugid = NULL;) +- USE_FEATURE_HTTPD_SETUID(struct bb_uidgid_t ugid;) +- USE_FEATURE_HTTPD_AUTH_MD5(const char *pass;) ++ IF_FEATURE_HTTPD_ENCODE_URL_STR(const char *url_for_encode;) ++ IF_FEATURE_HTTPD_SETUID(const char *s_ugid = NULL;) ++ IF_FEATURE_HTTPD_SETUID(struct bb_uidgid_t ugid;) ++ IF_FEATURE_HTTPD_AUTH_MD5(const char *pass;) + + INIT_G(); + +@@ -2299,16 +2311,16 @@ int httpd_main(int argc UNUSED_PARAM, ch + * If user gives relative path in -h, + * $SCRIPT_FILENAME will not be set. */ + opt = getopt32(argv, "c:d:h:" +- USE_FEATURE_HTTPD_ENCODE_URL_STR("e:") +- USE_FEATURE_HTTPD_BASIC_AUTH("r:") +- USE_FEATURE_HTTPD_AUTH_MD5("m:") +- USE_FEATURE_HTTPD_SETUID("u:") ++ IF_FEATURE_HTTPD_ENCODE_URL_STR("e:") ++ IF_FEATURE_HTTPD_BASIC_AUTH("r:") ++ IF_FEATURE_HTTPD_AUTH_MD5("m:") ++ IF_FEATURE_HTTPD_SETUID("u:") + "p:ifv", +- &configFile, &url_for_decode, &home_httpd +- USE_FEATURE_HTTPD_ENCODE_URL_STR(, &url_for_encode) +- USE_FEATURE_HTTPD_BASIC_AUTH(, &g_realm) +- USE_FEATURE_HTTPD_AUTH_MD5(, &pass) +- USE_FEATURE_HTTPD_SETUID(, &s_ugid) ++ &opt_c_configFile, &url_for_decode, &home_httpd ++ IF_FEATURE_HTTPD_ENCODE_URL_STR(, &url_for_encode) ++ IF_FEATURE_HTTPD_BASIC_AUTH(, &g_realm) ++ IF_FEATURE_HTTPD_AUTH_MD5(, &pass) ++ IF_FEATURE_HTTPD_SETUID(, &s_ugid) + , &bind_addr_or_port + , &verbose + ); +@@ -2375,7 +2387,7 @@ int httpd_main(int argc UNUSED_PARAM, ch + } + #endif + +- parse_conf(default_path_httpd_conf, FIRST_PARSE); ++ parse_conf(DEFAULT_PATH_HTTPD_CONF, FIRST_PARSE); + if (!(opt & OPT_INETD)) + signal(SIGHUP, sighup_handler); + diff --git a/package/busybox/busybox-1.14.1-telnetd.patch b/package/busybox/busybox-1.14.1-telnetd.patch new file mode 100644 index 0000000000..099e5eb839 --- /dev/null +++ b/package/busybox/busybox-1.14.1-telnetd.patch @@ -0,0 +1,85 @@ +diff -urpN busybox-1.14.1/networking/telnetd.c busybox-1.14.1-telnetd/networking/telnetd.c +--- busybox-1.14.1/networking/telnetd.c 2009-05-27 18:00:23.000000000 +0200 ++++ busybox-1.14.1-telnetd/networking/telnetd.c 2009-06-09 22:54:06.000000000 +0200 +@@ -199,9 +199,17 @@ static size_t iac_safe_write(int fd, con + return total + rc; + } + ++/* Must match getopt32 string */ ++enum { ++ OPT_WATCHCHILD = (1 << 2), /* -K */ ++ OPT_INETD = (1 << 3) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -i */ ++ OPT_PORT = (1 << 4) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -p */ ++ OPT_FOREGROUND = (1 << 6) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -F */ ++}; ++ + static struct tsession * + make_new_session( +- USE_FEATURE_TELNETD_STANDALONE(int sock) ++ USE_FEATURE_TELNETD_STANDALONE(int master_fd, int sock) + SKIP_FEATURE_TELNETD_STANDALONE(void) + ) { + const char *login_argv[2]; +@@ -288,9 +296,29 @@ make_new_session( + /* Restore default signal handling ASAP */ + bb_signals((1 << SIGCHLD) + (1 << SIGPIPE), SIG_DFL); + ++#if ENABLE_FEATURE_TELNETD_STANDALONE ++ if (!(option_mask32 & OPT_INETD)) { ++ struct tsession *tp = sessions; ++ while (tp) { ++ close(tp->ptyfd); ++ close(tp->sockfd_read); ++ /* sockfd_write == sockfd_read for standalone telnetd */ ++ /*close(tp->sockfd_write);*/ ++ tp = tp->next; ++ } ++ } ++#endif ++ + /* Make new session and process group */ + setsid(); + ++ close(fd); ++#if ENABLE_FEATURE_TELNETD_STANDALONE ++ close(sock); ++ if (master_fd >= 0) ++ close(master_fd); ++#endif ++ + /* Open the child's side of the tty. */ + /* NB: setsid() disconnects from any previous ctty's. Therefore + * we must open child's side of the tty AFTER setsid! */ +@@ -329,14 +357,6 @@ make_new_session( + _exit(EXIT_FAILURE); /*bb_perror_msg_and_die("execv %s", loginpath);*/ + } + +-/* Must match getopt32 string */ +-enum { +- OPT_WATCHCHILD = (1 << 2), /* -K */ +- OPT_INETD = (1 << 3) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -i */ +- OPT_PORT = (1 << 4) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -p */ +- OPT_FOREGROUND = (1 << 6) * ENABLE_FEATURE_TELNETD_STANDALONE, /* -F */ +-}; +- + #if ENABLE_FEATURE_TELNETD_STANDALONE + + static void +@@ -465,7 +485,7 @@ int telnetd_main(int argc UNUSED_PARAM, + + #if ENABLE_FEATURE_TELNETD_STANDALONE + if (IS_INETD) { +- sessions = make_new_session(0); ++ sessions = make_new_session(-1, 0); + if (!sessions) /* pty opening or vfork problem, exit */ + return 1; /* make_new_session prints error message */ + } else { +@@ -553,7 +573,7 @@ int telnetd_main(int argc UNUSED_PARAM, + if (fd < 0) + goto again; + /* Create a new session and link it into our active list */ +- new_ts = make_new_session(fd); ++ new_ts = make_new_session(master_fd, fd); + if (new_ts) { + new_ts->next = sessions; + sessions = new_ts; -- 2.30.2