httpd: reduce ifdef forest. comment out redundant PATH setting
[oweals/busybox.git] / networking / ifupdown.c
index 9841b5903c7735ea60e065180ae7b1974a12b4fd..ede2f8997a46980e09a8959ab92dec3cac79500b 100644 (file)
@@ -89,7 +89,6 @@ struct interfaces_file_t
        struct mapping_defn_t *mappings;
 };
 
-static unsigned option_mask;
 #define OPTION_STR "anvf" USE_FEATURE_IFUPDOWN_MAPPING("m") "i:"
 enum {
        OPT_do_all = 0x1,
@@ -98,14 +97,16 @@ enum {
        OPT_force = 0x8,
        OPT_no_mappings = 0x10,
 };
-#define DO_ALL (option_mask & OPT_do_all)
-#define NO_ACT (option_mask & OPT_no_act)
-#define VERBOSE (option_mask & OPT_verbose)
-#define FORCE (option_mask & OPT_force)
-#define NO_MAPPINGS (option_mask & OPT_no_mappings)
+#define DO_ALL (option_mask32 & OPT_do_all)
+#define NO_ACT (option_mask32 & OPT_no_act)
+#define VERBOSE (option_mask32 & OPT_verbose)
+#define FORCE (option_mask32 & OPT_force)
+#define NO_MAPPINGS (option_mask32 & OPT_no_mappings)
 
 static char **__myenviron;
 
+static char *startup_PATH;
+
 #if ENABLE_FEATURE_IFUPDOWN_IPV4 || ENABLE_FEATURE_IFUPDOWN_IPV6
 
 #ifdef CONFIG_FEATURE_IFUPDOWN_IP
@@ -132,7 +133,7 @@ static int count_netmask_bits(char *dotted_quad)
 }
 #endif
 
-static void addstr(char **buf, size_t *len, size_t *pos, char *str, size_t str_length)
+static void addstr(char **buf, size_t *len, size_t *pos, const char *str, size_t str_length)
 {
        if (*pos + str_length >= *len) {
                char *newbuf;
@@ -149,7 +150,7 @@ static void addstr(char **buf, size_t *len, size_t *pos, char *str, size_t str_l
        (*buf)[*pos] = '\0';
 }
 
-static int strncmpz(char *l, char *r, size_t llen)
+static int strncmpz(const char *l, const char *r, size_t llen)
 {
        int i = strncmp(l, r, llen);
 
@@ -160,7 +161,7 @@ static int strncmpz(char *l, char *r, size_t llen)
        }
 }
 
-static char *get_var(char *id, size_t idlen, struct interface_defn_t *ifd)
+static char *get_var(const char *id, size_t idlen, struct interface_defn_t *ifd)
 {
        int i;
 
@@ -187,7 +188,7 @@ static char *get_var(char *id, size_t idlen, struct interface_defn_t *ifd)
        return NULL;
 }
 
-static char *parse(char *command, struct interface_defn_t *ifd)
+static char *parse(const char *command, struct interface_defn_t *ifd)
 {
 
        char *result = NULL;
@@ -293,7 +294,7 @@ static char *parse(char *command, struct interface_defn_t *ifd)
 }
 
 /* execute() returns 1 for success and 0 for failure */
-static int execute(char *command, struct interface_defn_t *ifd, execfn *exec)
+static int execute(const char *command, struct interface_defn_t *ifd, execfn *exec)
 {
        char *out;
        int ret;
@@ -448,50 +449,70 @@ static int static_down(struct interface_defn_t *ifd, execfn *exec)
        return ((result == 2) ? 2 : 0);
 }
 
-static int execable(char *program)
+#ifndef CONFIG_APP_UDHCPC
+struct dhcp_client_t
 {
-       struct stat buf;
-       if (0 == stat(program, &buf)) {
-               if (S_ISREG(buf.st_mode) && (S_IXUSR & buf.st_mode)) {
-                       return(1);
-               }
-       }
-       return(0);
-}
+       const char *name;
+       const char *startcmd;
+       const char *stopcmd;
+};
+
+static const struct dhcp_client_t ext_dhcp_clients[] = {
+       { "udhcpc",
+               "udhcpc -R -n -p /var/run/udhcpc.%iface%.pid -i %iface% [[-H %hostname%]] [[-c %clientid%]] [[-s %script%]]",
+               "kill -TERM `cat /var/run/udhcpc.%iface%.pid` 2>/dev/null",
+       },
+       { "pump",
+               "pump -i %iface% [[-h %hostname%]] [[-l %leasehours%]]",
+               "pump -i %iface% -k",
+       },
+       { "dhclient",
+               "dhclient -pf /var/run/dhclient.%iface%.pid %iface%",
+               "kill -9 `cat /var/run/dhclient.%iface%.pid` 2>/dev/null",
+       },
+       { "dhcpcd",
+               "dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %clientid%]] [[-l %leasetime%]] %iface%",
+               "dhcpcd -k %iface%",
+       },
+};
+#endif
 
 static int dhcp_up(struct interface_defn_t *ifd, execfn *exec)
 {
-       if (execable("/sbin/udhcpc")) {
-               return( execute("udhcpc -n -p /var/run/udhcpc.%iface%.pid -i "
-                                       "%iface% [[-H %hostname%]] [[-c %clientid%]]", ifd, exec));
-       } else if (execable("/sbin/pump")) {
-               return( execute("pump -i %iface% [[-h %hostname%]] [[-l %leasehours%]]", ifd, exec));
-       } else if (execable("/sbin/dhclient")) {
-               return( execute("dhclient -pf /var/run/dhclient.%iface%.pid %iface%", ifd, exec));
-       } else if (execable("/sbin/dhcpcd")) {
-               return( execute("dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %clientid%]] "
-                                       "[[-l %leasetime%]] %iface%", ifd, exec));
+#ifdef CONFIG_APP_UDHCPC
+       return execute("udhcpc -R -n -p /var/run/udhcpc.%iface%.pid "
+                       "-i %iface% [[-H %hostname%]] [[-c %clientid%]] [[-s %script%]]",
+                       ifd, exec);
+#else
+       int i, nclients = sizeof(ext_dhcp_clients) / sizeof(ext_dhcp_clients[0]);
+       for (i = 0; i < nclients; i++) {
+               if (exists_execable(ext_dhcp_clients[i].name))
+                       return execute(ext_dhcp_clients[i].startcmd, ifd, exec);
        }
-       return(0);
+       bb_error_msg("no dhcp clients found");
+       return 0;
+#endif
 }
 
 static int dhcp_down(struct interface_defn_t *ifd, execfn *exec)
 {
-       int result = 0;
-       if (execable("/sbin/udhcpc")) {
-               /* SIGUSR2 forces udhcpc to release the current lease and go inactive,
-                * and SIGTERM causes udhcpc to exit.  Signals are queued and processed
-                * sequentially so we don't need to sleep */
-               result = execute("kill -USR2 `cat /var/run/udhcpc.%iface%.pid` 2>/dev/null", ifd, exec);
-               result += execute("kill -TERM `cat /var/run/udhcpc.%iface%.pid` 2>/dev/null", ifd, exec);
-       } else if (execable("/sbin/pump")) {
-               result = execute("pump -i %iface% -k", ifd, exec);
-       } else if (execable("/sbin/dhclient")) {
-               result = execute("kill -9 `cat /var/run/dhclient.%iface%.pid` 2>/dev/null", ifd, exec);
-       } else if (execable("/sbin/dhcpcd")) {
-               result = execute("dhcpcd -k %iface%", ifd, exec);
+#ifdef CONFIG_APP_UDHCPC
+       return execute("kill -TERM "
+                      "`cat /var/run/udhcpc.%iface%.pid` 2>/dev/null", ifd, exec);
+#else
+       int i, nclients = sizeof(ext_dhcp_clients) / sizeof(ext_dhcp_clients[0]);
+       for (i = 0; i < nclients; i++) {
+               if (exists_execable(ext_dhcp_clients[i].name))
+                       return execute(ext_dhcp_clients[i].stopcmd, ifd, exec);
        }
-       return (result || static_down(ifd, exec));
+       bb_error_msg("no dhcp clients found, using static interface shutdown");
+       return static_down(ifd, exec);
+#endif
+}
+
+static int manual_up_down(struct interface_defn_t *ifd, execfn *exec)
+{
+       return 1;
 }
 
 static int bootp_up(struct interface_defn_t *ifd, execfn *exec)
@@ -524,6 +545,7 @@ static int wvdial_down(struct interface_defn_t *ifd, execfn *exec)
 }
 
 static const struct method_t methods[] = {
+       { "manual", manual_up_down, manual_up_down, },
        { "wvdial", wvdial_up, wvdial_down, },
        { "ppp", ppp_up, ppp_down, },
        { "static", static_up, static_down, },
@@ -550,10 +572,7 @@ static char *next_word(char **buf)
        }
 
        /* Skip over leading whitespace */
-       word = *buf;
-       while (isspace(*word)) {
-               ++word;
-       }
+       word = skip_whitespace(*buf);
 
        /* Skip over comments */
        if (*word == '#') {
@@ -627,7 +646,7 @@ static struct interfaces_file_t *read_interfaces(const char *filename)
 
        f = xfopen(filename, "r");
 
-       while ((buf = bb_get_chomped_line_from_file(f)) != NULL) {
+       while ((buf = xmalloc_getline(f)) != NULL) {
                char *buf_ptr = buf;
 
                firstword = next_word(&buf_ptr);
@@ -690,9 +709,7 @@ static struct interfaces_file_t *read_interfaces(const char *filename)
                        }
 
                        /* ship any trailing whitespace */
-                       while (isspace(*buf_ptr)) {
-                               ++buf_ptr;
-                       }
+                       buf_ptr = skip_whitespace(buf_ptr);
 
                        if (buf_ptr[0] != '\0') {
                                bb_error_msg("too many parameters \"%s\"", buf);
@@ -875,17 +892,15 @@ static void set_environ(struct interface_defn_t *iface, const char *mode)
        *(environend++) = setlocalenv("%s=%s", "ADDRFAM", iface->address_family->name);
        *(environend++) = setlocalenv("%s=%s", "METHOD", iface->method->name);
        *(environend++) = setlocalenv("%s=%s", "MODE", mode);
-       /* FIXME: we must not impose our own PATH, it is admin's job!
-          Use PATH from parent env */
-       *(environend++) = setlocalenv("%s=%s", "PATH", "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin");
+       *(environend++) = setlocalenv("%s=%s", "PATH", startup_PATH);
 }
 
 static int doit(char *str)
 {
-       if (option_mask & (OPT_no_act|OPT_verbose)) {
+       if (option_mask32 & (OPT_no_act|OPT_verbose)) {
                puts(str);
        }
-       if (!(option_mask & OPT_no_act)) {
+       if (!(option_mask32 & OPT_no_act)) {
                pid_t child;
                int status;
 
@@ -1079,9 +1094,8 @@ int ifupdown_main(int argc, char **argv)
        llist_t *target_list = NULL;
        const char *interfaces = "/etc/network/interfaces";
        int any_failures = 0;
-       int i;
 
-       if (bb_applet_name[2] == 'u') {
+       if (applet_name[2] == 'u') {
                /* ifup command */
                cmds = iface_up;
        } else {
@@ -1089,7 +1103,7 @@ int ifupdown_main(int argc, char **argv)
                cmds = iface_down;
        }
 
-       option_mask = bb_getopt_ulflags(argc, argv, OPTION_STR, &interfaces);
+       getopt32(argc, argv, OPTION_STR, &interfaces);
        if (argc - optind > 0) {
                if (DO_ALL) bb_show_usage();
        } else
@@ -1103,6 +1117,9 @@ int ifupdown_main(int argc, char **argv)
                exit(EXIT_FAILURE);
        }
 
+       startup_PATH = getenv("PATH");
+       if (!startup_PATH) startup_PATH = "";
+
        /* Create a list of interfaces to work on */
        if (DO_ALL) {
                if (cmds == iface_up) {
@@ -1165,6 +1182,7 @@ int ifupdown_main(int argc, char **argv)
                        struct mapping_defn_t *currmap;
 
                        for (currmap = defn->mappings; currmap; currmap = currmap->next) {
+                               int i;
                                for (i = 0; i < currmap->n_matches; i++) {
                                        if (fnmatch(currmap->match[i], liface, 0) != 0)
                                                continue;