post 1.14.1 fixes; bump version to 1.14.2 1_14_2
authorDenys Vlasenko <vda.linux@googlemail.com>
Sun, 21 Jun 2009 22:47:18 +0000 (00:47 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sun, 21 Jun 2009 22:47:18 +0000 (00:47 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
12 files changed:
Makefile
coreutils/readlink.c
include/libbb.h
include/usage.h
libbb/simplify_path.c
modutils/modprobe-small.c
modutils/modprobe.c
modutils/modutils.c
networking/ftpd.c
networking/httpd.c
networking/telnetd.c
shell/ash.c

index e87d18d2ae41fdb434cff6c36bda02cb9bfb4316..7dea756a156f044fd15deb156a6ef40bf7ba1710 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,6 @@
 VERSION = 1
 PATCHLEVEL = 14
-SUBLEVEL = 1
+SUBLEVEL = 2
 EXTRAVERSION =
 NAME = Unnamed
 
index 721fd8597a422cc8bfb522a438fb81eb8d93f7cb..bcf352e74b50e43a9d7bdb687c800cde747e153e 100644 (file)
@@ -6,9 +6,31 @@
  *
  * Licensed under GPL v2 or later, see file LICENSE in this tarball for details.
  */
-
 #include "libbb.h"
 
+/*
+ * # readlink --version
+ * readlink (GNU coreutils) 6.10
+ * # readlink --help
+ *   -f, --canonicalize
+ *      canonicalize by following every symlink in
+ *      every component of the given name recursively;
+ *      all but the last component must exist
+ *   -e, --canonicalize-existing
+ *      canonicalize by following every symlink in
+ *      every component of the given name recursively,
+ *      all components must exist
+ *   -m, --canonicalize-missing
+ *      canonicalize by following every symlink in
+ *      every component of the given name recursively,
+ *      without requirements on components existence
+ *   -n, --no-newline              do not output the trailing newline
+ *   -q, --quiet, -s, --silent     suppress most error messages
+ *   -v, --verbose                 report error messages
+ *
+ * bbox supports: -f -n -v (fully), -q -s (accepts but ignores)
+ */
+
 int readlink_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int readlink_main(int argc UNUSED_PARAM, char **argv)
 {
@@ -20,7 +42,7 @@ int readlink_main(int argc UNUSED_PARAM, char **argv)
                unsigned opt;
                /* We need exactly one non-option argument.  */
                opt_complementary = "=1";
-               opt = getopt32(argv, "f");
+               opt = getopt32(argv, "fnvsq");
                fname = argv[optind];
        )
        SKIP_FEATURE_READLINK_FOLLOW(
@@ -30,9 +52,10 @@ int readlink_main(int argc UNUSED_PARAM, char **argv)
        )
 
        /* compat: coreutils readlink reports errors silently via exit code */
-       logmode = LOGMODE_NONE;
+       if (!(opt & 4)) /* not -v */
+               logmode = LOGMODE_NONE;
 
-       if (opt) {
+       if (opt & 1) { /* -f */
                buf = realpath(fname, pathbuf);
        } else {
                buf = xmalloc_readlink_or_warn(fname);
@@ -40,7 +63,7 @@ int readlink_main(int argc UNUSED_PARAM, char **argv)
 
        if (!buf)
                return EXIT_FAILURE;
-       puts(buf);
+       printf((opt & 2) ? "%s" : "%s\n", buf);
 
        if (ENABLE_FEATURE_CLEAN_UP && !opt)
                free(buf);
index 2868eba036c466399e6b73ef983690d61d80d10e..bedf659b7ec7ce7713ae8843ca1f5125763e6647 100644 (file)
@@ -1099,6 +1099,8 @@ const char *get_signame(int number) FAST_FUNC;
 void print_signames(void) FAST_FUNC;
 
 char *bb_simplify_path(const char *path) FAST_FUNC;
+/* Returns ptr to NUL */
+char *bb_simplify_abs_path_inplace(char *path) FAST_FUNC;
 
 #define FAIL_DELAY 3
 extern void bb_do_delay(int seconds) FAST_FUNC;
index d3bf7849e0f95099a03f576dab68e82a73234218..bfacc5611d8412c004e99ee79c0619db88d06f91 100644 (file)
        "files do not block on disk I/O"
 
 #define readlink_trivial_usage \
-       USE_FEATURE_READLINK_FOLLOW("[-f] ") "FILE"
+       USE_FEATURE_READLINK_FOLLOW("[-fnv] ") "FILE"
 #define readlink_full_usage "\n\n" \
        "Display the value of a symlink" \
        USE_FEATURE_READLINK_FOLLOW( "\n" \
      "\nOptions:" \
-     "\n       -f      Canonicalize by following all symlinks") \
+     "\n       -f      Canonicalize by following all symlinks" \
+     "\n       -n      Don't add newline" \
+     "\n       -v      Verbose" \
+       ) \
 
 #define readprofile_trivial_usage \
        "[OPTIONS]..."
index 367f1f04d9ca6773b6ac7258a04590b8efeef04f..f80e3e8a5eef53ddeba2cd365bbd39506cfea8f9 100644 (file)
@@ -6,22 +6,13 @@
  *
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
-
 #include "libbb.h"
 
-char* FAST_FUNC bb_simplify_path(const char *path)
+char* FAST_FUNC bb_simplify_abs_path_inplace(char *start)
 {
-       char *s, *start, *p;
+       char *s, *p;
 
-       if (path[0] == '/')
-               start = xstrdup(path);
-       else {
-               s = xrealloc_getcwd_or_warn(NULL);
-               start = concat_path_file(s, path);
-               free(s);
-       }
        p = s = start;
-
        do {
                if (*p == '/') {
                        if (*s == '/') {        /* skip duplicate (or initial) slash */
@@ -47,7 +38,22 @@ char* FAST_FUNC bb_simplify_path(const char *path)
        if ((p == start) || (*p != '/')) {      /* not a trailing slash */
                ++p;                                    /* so keep last character */
        }
-       *p = 0;
+       *p = '\0';
+       return p;
+}
+
+char* FAST_FUNC bb_simplify_path(const char *path)
+{
+       char *s, *p;
+
+       if (path[0] == '/')
+               s = xstrdup(path);
+       else {
+               p = xrealloc_getcwd_or_warn(NULL);
+               s = concat_path_file(p, path);
+               free(p);
+       }
 
-       return start;
+       bb_simplify_abs_path_inplace(s);
+       return s;
 }
index 6eb950f32ea8103136a39a020a0ba8c46ce663cc..6ee0164c25b1c6a041a8b418f72d10918ec3ab1c 100644 (file)
@@ -656,7 +656,7 @@ depmod -[aA] [-n -e -v -q -V -r -u]
       [-b basedirectory] [forced_version]
 depmod [-n -e -v -q -r -u] [-F kernelsyms] module1.ko module2.ko ...
 If no arguments (except options) are given, "depmod -a" is assumed.
-depmod will output a dependancy list suitable for the modprobe utility.
+depmod will output a dependency list suitable for the modprobe utility.
 Options:
     -a, --all           Probe all modules
     -A, --quick         Only does the work if there's a new module
index 0339ebb405f673457a88bcfabea705327bb39daa..310eebc5f7ed6388bcc8f78f80be6c23332d04cc 100644 (file)
@@ -8,12 +8,17 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
+/* Note that unlike older versions of modules.dep/depmod (busybox and m-i-t),
+ * we expect the full dependency list to be specified in modules.dep.  Older
+ * versions would only export the direct dependency list.
+ */
+
 #include "libbb.h"
 #include "modutils.h"
 #include <sys/utsname.h>
 #include <fnmatch.h>
 
-//#define DBG(...) bb_error_msg(__VA_ARGS__)
+//#define DBG(fmt, ...) bb_error_msg("%s: " fmt, __func__, ## __VA_ARGS__)
 #define DBG(...) ((void)0)
 
 #define MODULE_FLAG_LOADED             0x0001
@@ -116,6 +121,7 @@ static void add_probe(const char *name)
                return;
        }
 
+       DBG("queuing %s", name);
        m->probed_name = name;
        m->flags |= MODULE_FLAG_NEED_DEPS;
        llist_add_to_end(&G.probes, m);
@@ -205,9 +211,10 @@ static int read_config(const char *path)
 
 static int do_modprobe(struct module_entry *m)
 {
-       struct module_entry *m2;
+       struct module_entry *m2 = m2; /* for compiler */
        char *fn, *options;
-       int rc = -1;
+       int rc, first;
+       llist_t *l;
 
        if (!(m->flags & MODULE_FLAG_FOUND_IN_MODDEP)) {
                DBG("skipping %s, not found in modules.dep", m->modname);
@@ -218,13 +225,25 @@ static int do_modprobe(struct module_entry *m)
        if (!(option_mask32 & MODPROBE_OPT_REMOVE))
                m->deps = llist_rev(m->deps);
 
+       for (l = m->deps; l != NULL; l = l->link)
+               DBG("dep: %s", l->data);
+
+       first = 1;
        rc = 0;
        while (m->deps && rc == 0) {
                fn = llist_pop(&m->deps);
                m2 = get_or_add_modentry(fn);
                if (option_mask32 & MODPROBE_OPT_REMOVE) {
-                       if (bb_delete_module(m->modname, O_EXCL) != 0)
-                               rc = errno;
+                       if (m2->flags & MODULE_FLAG_LOADED) {
+                               if (bb_delete_module(m2->modname, O_EXCL) != 0) {
+                                       if (first)
+                                               rc = errno;
+                               } else {
+                                       m2->flags &= ~MODULE_FLAG_LOADED;
+                               }
+                       }
+                       /* do not error out if *deps* fail to unload */
+                       first = 0;
                } else if (!(m2->flags & MODULE_FLAG_LOADED)) {
                        options = m2->options;
                        m2->options = NULL;
@@ -242,11 +261,10 @@ static int do_modprobe(struct module_entry *m)
                free(fn);
        }
 
-//FIXME: what if rc < 0?
-       if (rc > 0 && !(option_mask32 & INSMOD_OPT_SILENT)) {
+       if (rc && !(option_mask32 & INSMOD_OPT_SILENT)) {
                bb_error_msg("failed to %sload module %s: %s",
                        (option_mask32 & MODPROBE_OPT_REMOVE) ? "un" : "",
-                       m->probed_name ? m->probed_name : m->modname,
+                       m2->probed_name ? m2->probed_name : m2->modname,
                        moderror(rc)
                );
        }
@@ -294,7 +312,8 @@ static void load_modules_dep(void)
                        llist_add_to(&m->deps, xstrdup(tokens[0]));
                        if (tokens[1])
                                string_to_llist(tokens[1], &m->deps, " ");
-               }
+               } else
+                       DBG("skipping dep line");
        }
        config_close(p);
 }
@@ -344,10 +363,12 @@ int modprobe_main(int argc UNUSED_PARAM, char **argv)
        if (opt & (MODPROBE_OPT_INSERT_ALL | MODPROBE_OPT_REMOVE)) {
                /* Each argument is a module name */
                do {
+                       DBG("adding module %s", *argv);
                        add_probe(*argv++);
                } while (*argv);
        } else {
                /* First argument is module name, rest are parameters */
+               DBG("probing just module %s", *argv);
                add_probe(argv[0]);
                G.cmdline_mopts = parse_cmdline_module_options(argv);
        }
index 0f6cb0f2db0c799a11fa41f649be2e2ffe90cd64..f437a9829c66b1cd95f64712871b155c031e629b 100644 (file)
@@ -57,7 +57,7 @@ char * FAST_FUNC filename2modname(const char *filename, char *modname)
        from = bb_get_last_path_component_nostrip(filename);
        for (i = 0; i < (MODULE_NAME_LEN-1) && from[i] != '\0' && from[i] != '.'; i++)
                modname[i] = (from[i] == '-') ? '_' : from[i];
-       modname[i] = 0;
+       modname[i] = '\0';
 
        return modname;
 }
index ac68961170b1e709466139439b114cccc91f8f12..e85e4f8ea16d626835041b76d9b63480fc7944fb 100644 (file)
@@ -1320,6 +1320,8 @@ int ftpd_main(int argc UNUSED_PARAM, char **argv)
                                handle_appe();
                        else if (cmdval == const_STOU) /* "store unique" */
                                handle_stou();
+                       else
+                               goto bad_cmd;
                }
 #endif
 #if 0
@@ -1340,6 +1342,9 @@ int ftpd_main(int argc UNUSED_PARAM, char **argv)
                         * (doesn't necessarily mean "we must support them")
                         * foo 1.2.3: XXXX - comment
                         */
+#if ENABLE_FEATURE_FTP_WRITE
+ bad_cmd:
+#endif
                        cmdio_write_raw(STR(FTP_BADCMD)" Unknown command\r\n");
                }
        }
index 6bf103c56fe6b555eed241a03bda720d7ff6381d..de4fb9b39beafcc17e1f7a155b42641b899d2dfe 100644 (file)
@@ -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:
  * 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 */
 
 #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;
 
@@ -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  )
@@ -452,14 +452,6 @@ static int scan_ip_mask(const char *str, unsigned *ipp, unsigned *maskp)
 /*
  * 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, unsigned *ipp, unsigned *maskp)
  * 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, int flag)
                }
 
 #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, int flag)
 #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, int flag)
                        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, int flag)
                        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(const char *url, int what)
                        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(const char *url, int what)
                        }
                }
        }
+
+       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 */
@@ -2031,8 +2043,8 @@ static void handle_incoming_and_exit(const len_and_sockaddr *fromAddr)
        /* 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(const len_and_sockaddr *fromAddr)
                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,7 +2257,7 @@ 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 {
@@ -2304,7 +2316,7 @@ int httpd_main(int argc UNUSED_PARAM, char **argv)
                        USE_FEATURE_HTTPD_AUTH_MD5("m:")
                        USE_FEATURE_HTTPD_SETUID("u:")
                        "p:ifv",
-                       &configFile, &url_for_decode, &home_httpd
+                       &opt_c_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)
@@ -2375,7 +2387,7 @@ int httpd_main(int argc UNUSED_PARAM, char **argv)
        }
 #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);
 
index ccf32892508a45397dae392f4a8e79f66cf05153..4c5ea3ab39a8319aa424ea3b3a236d2e0714ebd8 100644 (file)
@@ -199,9 +199,17 @@ static size_t iac_safe_write(int fd, const char *buf, size_t count)
        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, char **argv)
 
 #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, char **argv)
                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;
index 4acc30110b99ba1765813ddf7c6316a760712536..4981f4ce0cc520662517568eb5b2b41c3e2eb293 100644 (file)
@@ -11909,7 +11909,8 @@ find_dot_file(char *name)
                         */
                        return fullname;
                }
-               stunalloc(fullname);
+               if (fullname != name)
+                       stunalloc(fullname);
        }
 
        /* not found in the PATH */