summaryrefslogtreecommitdiffstats
path: root/package/busybox/busybox-1.14.1-httpd.patch
diff options
context:
space:
mode:
authorPeter Korsgaard <jacmet@sunsite.dk>2009-06-11 12:34:01 +0200
committerPeter Korsgaard <jacmet@sunsite.dk>2009-06-11 12:34:01 +0200
commite491611316943fb5a44c94f27efecb9a9117a3c5 (patch)
tree3d57c94548ddb379670edf0a81f44c5ab8f2af1d /package/busybox/busybox-1.14.1-httpd.patch
parent6fac68300eeddb0a3eec2e745b208e4f39ff5811 (diff)
downloadbuildroot-novena-e491611316943fb5a44c94f27efecb9a9117a3c5.tar.gz
buildroot-novena-e491611316943fb5a44c94f27efecb9a9117a3c5.zip
busybox: additional 1.14.1 fixes
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Diffstat (limited to 'package/busybox/busybox-1.14.1-httpd.patch')
-rw-r--r--package/busybox/busybox-1.14.1-httpd.patch733
1 files changed, 733 insertions, 0 deletions
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 000000000..ff53f0d83
--- /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 "<Hello World>"` # encode as "&#60Hello&#32World&#62"
+ * 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 <sys/sendfile.h>
+ #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);
+