accidentally applied wrong (old) patch, fixing up...
[oweals/busybox.git] / networking / ifupdown.c
index 99b1c59d99f07d29a0ab7b096f10f60a78c26e70..adbc37e43e4daad06541ff63cf03ef13ad90f218 100644 (file)
 #define EUNDEFVAR   10002
 #define EUNBALPER   10000
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
 #define MAX_INTERFACE_LENGTH 10
 #endif
 
-#define debug_noise(fmt, args...)
+#define debug_noise(args...) /*fprintf(stderr, args)*/
 
 /* Forward declaration */
 struct interface_defn_t;
 
-typedef int (execfn)(char *command);
+typedef int execfn(char *command);
 
 struct method_t
 {
@@ -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,34 +97,35 @@ 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 **my_environ;
 
 static char *startup_PATH;
 
 #if ENABLE_FEATURE_IFUPDOWN_IPV4 || ENABLE_FEATURE_IFUPDOWN_IPV6
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
 
-static unsigned int count_bits(unsigned int a)
+static unsigned count_bits(unsigned a)
 {
-       unsigned int result;
+       unsigned result;
        result = (a & 0x55) + ((a >> 1) & 0x55);
        result = (result & 0x33) + ((result >> 2) & 0x33);
-       return ((result & 0x0F) + ((result >> 4) & 0x0F));
+       return (result & 0x0F) + ((result >> 4) & 0x0F);
 }
 
 static int count_netmask_bits(char *dotted_quad)
 {
-       unsigned int result, a, b, c, d;
+       unsigned result, a, b, c, d;
        /* Found a netmask...  Check if it is dotted quad */
        if (sscanf(dotted_quad, "%u.%u.%u.%u", &a, &b, &c, &d) != 4)
                return -1;
+       // FIXME: will be confused by e.g. 255.0.255.0
        result = count_bits(a);
        result += count_bits(b);
        result += count_bits(c);
@@ -134,93 +134,84 @@ 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 **bufp, const char *str, size_t str_length)
 {
-       if (*pos + str_length >= *len) {
-               char *newbuf;
-
-               newbuf = xrealloc(*buf, *len * 2 + str_length + 1);
-               *buf = newbuf;
-               *len = *len * 2 + str_length + 1;
-       }
-
-       while (str_length-- >= 1) {
-               (*buf)[(*pos)++] = *str;
-               str++;
-       }
-       (*buf)[*pos] = '\0';
+       /* xasprintf trick will be smaller, but we are often
+        * called with str_length == 1 - don't want to have
+        * THAT much of malloc/freeing! */
+       char *buf = *bufp;
+       int len = (buf ? strlen(buf) : 0);
+       str_length++;
+       buf = xrealloc(buf, len + str_length);
+       /* copies at most str_length-1 chars! */
+       safe_strncpy(buf + len, str, str_length);
+       *bufp = buf;
 }
 
-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);
 
-       if (i == 0) {
+       if (i == 0)
                return -r[llen];
-       } else {
-               return i;
-       }
+       return i;
 }
 
-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;
 
        if (strncmpz(id, "iface", idlen) == 0) {
                char *result;
                static char label_buf[20];
-               strncpy(label_buf, ifd->iface, 19);
-               label_buf[19]=0;
+               safe_strncpy(label_buf, ifd->iface, sizeof(label_buf));
                result = strchr(label_buf, ':');
                if (result) {
-                       *result=0;
+                       *result = '\0';
                }
                return label_buf;
-       } else if (strncmpz(id, "label", idlen) == 0) {
+       }
+       if (strncmpz(id, "label", idlen) == 0) {
                return ifd->iface;
-       } else {
-               for (i = 0; i < ifd->n_options; i++) {
-                       if (strncmpz(id, ifd->option[i].name, idlen) == 0) {
-                               return ifd->option[i].value;
-                       }
+       }
+       for (i = 0; i < ifd->n_options; i++) {
+               if (strncmpz(id, ifd->option[i].name, idlen) == 0) {
+                       return ifd->option[i].value;
                }
        }
-
        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;
-       size_t pos = 0, len = 0;
        size_t old_pos[MAX_OPT_DEPTH] = { 0 };
        int okay[MAX_OPT_DEPTH] = { 1 };
        int opt_depth = 1;
+       char *result = NULL;
 
        while (*command) {
                switch (*command) {
                default:
-                       addstr(&result, &len, &pos, command, 1);
+                       addstr(&result, command, 1);
                        command++;
                        break;
                case '\\':
                        if (command[1]) {
-                               addstr(&result, &len, &pos, command + 1, 1);
+                               addstr(&result, command + 1, 1);
                                command += 2;
                        } else {
-                               addstr(&result, &len, &pos, command, 1);
+                               addstr(&result, command, 1);
                                command++;
                        }
                        break;
                case '[':
                        if (command[1] == '[' && opt_depth < MAX_OPT_DEPTH) {
-                               old_pos[opt_depth] = pos;
+                               old_pos[opt_depth] = result ? strlen(result) : 0;
                                okay[opt_depth] = 1;
                                opt_depth++;
                                command += 2;
                        } else {
-                               addstr(&result, &len, &pos, "[", 1);
+                               addstr(&result, "[", 1);
                                command++;
                        }
                        break;
@@ -228,12 +219,11 @@ static char *parse(char *command, struct interface_defn_t *ifd)
                        if (command[1] == ']' && opt_depth > 1) {
                                opt_depth--;
                                if (!okay[opt_depth]) {
-                                       pos = old_pos[opt_depth];
-                                       result[pos] = '\0';
+                                       result[old_pos[opt_depth]] = '\0';
                                }
                                command += 2;
                        } else {
-                               addstr(&result, &len, &pos, "]", 1);
+                               addstr(&result, "]", 1);
                                command++;
                        }
                        break;
@@ -253,18 +243,17 @@ static char *parse(char *command, struct interface_defn_t *ifd)
                                varvalue = get_var(command, nextpercent - command, ifd);
 
                                if (varvalue) {
-                                       addstr(&result, &len, &pos, varvalue, strlen(varvalue));
+                                       addstr(&result, varvalue, strlen(varvalue));
                                } else {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
                                        /* Sigh...  Add a special case for 'ip' to convert from
                                         * dotted quad to bit count style netmasks.  */
-                                       if (strncmp(command, "bnmask", 6)==0) {
-                                               int res;
+                                       if (strncmp(command, "bnmask", 6) == 0) {
+                                               unsigned res;
                                                varvalue = get_var("netmask", 7, ifd);
-                                               if (varvalue && (res=count_netmask_bits(varvalue)) > 0) {
-                                                       char argument[255];
-                                                       sprintf(argument, "%d", res);
-                                                       addstr(&result, &len, &pos, argument, strlen(argument));
+                                               if (varvalue && (res = count_netmask_bits(varvalue)) > 0) {
+                                                       const char *argument = utoa(res);
+                                                       addstr(&result, argument, strlen(argument));
                                                        command = nextpercent + 1;
                                                        break;
                                                }
@@ -295,16 +284,18 @@ 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;
 
        out = parse(command, ifd);
        if (!out) {
+               /* parse error? */
                return 0;
        }
-       ret = (*exec)(out);
+       /* out == "": parsed ok but not all needed variables known, skip */
+       ret = out[0] ? (*exec)(out) : 1;
 
        free(out);
        if (ret != 1) {
@@ -314,10 +305,10 @@ static int execute(char *command, struct interface_defn_t *ifd, execfn *exec)
 }
 #endif
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPV6
+#if ENABLE_FEATURE_IFUPDOWN_IPV6
 static int loopback_up6(struct interface_defn_t *ifd, execfn *exec)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        int result;
        result = execute("ip addr add ::1 dev %iface%", ifd, exec);
        result += execute("ip link set %iface% up", ifd, exec);
@@ -329,7 +320,7 @@ static int loopback_up6(struct interface_defn_t *ifd, execfn *exec)
 
 static int loopback_down6(struct interface_defn_t *ifd, execfn *exec)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        return execute("ip link set %iface% down", ifd, exec);
 #else
        return execute("ifconfig %iface% del ::1", ifd, exec);
@@ -339,36 +330,37 @@ static int loopback_down6(struct interface_defn_t *ifd, execfn *exec)
 static int static_up6(struct interface_defn_t *ifd, execfn *exec)
 {
        int result;
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
-       result = execute("ip addr add %address%/%netmask% dev %iface% [[label %label%]]", ifd, exec);
-       result += execute("ip link set [[mtu %mtu%]] [[address %hwaddress%]] %iface% up", ifd, exec);
-       result += execute("[[ ip route add ::/0 via %gateway% ]]", ifd, exec);
+#if ENABLE_FEATURE_IFUPDOWN_IP
+       result = execute("ip addr add %address%/%netmask% dev %iface%[[ label %label%]]", ifd, exec);
+       result += execute("ip link set[[ mtu %mtu%]][[ address %hwaddress%]] %iface% up", ifd, exec);
+       /* Was: "[[ ip ....%gateway% ]]". Removed extra spaces w/o checking */
+       result += execute("[[ip route add ::/0 via %gateway%]]", ifd, exec);
 #else
-       result = execute("ifconfig %iface% [[media %media%]] [[hw %hwaddress%]] [[mtu %mtu%]] up", ifd, exec);
+       result = execute("ifconfig %iface%[[ media %media%]][[ hw %hwaddress%]][[ mtu %mtu%]] up", ifd, exec);
        result += execute("ifconfig %iface% add %address%/%netmask%", ifd, exec);
-       result += execute("[[ route -A inet6 add ::/0 gw %gateway% ]]", ifd, exec);
+       result += execute("[[route -A inet6 add ::/0 gw %gateway%]]", ifd, exec);
 #endif
        return ((result == 3) ? 3 : 0);
 }
 
 static int static_down6(struct interface_defn_t *ifd, execfn *exec)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        return execute("ip link set %iface% down", ifd, exec);
 #else
        return execute("ifconfig %iface% down", ifd, exec);
 #endif
 }
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
 static int v4tunnel_up(struct interface_defn_t *ifd, execfn *exec)
 {
        int result;
        result = execute("ip tunnel add %iface% mode sit remote "
-                       "%endpoint% [[local %local%]] [[ttl %ttl%]]", ifd, exec);
+                       "%endpoint%[[ local %local%]][[ ttl %ttl%]]", ifd, exec);
        result += execute("ip link set %iface% up", ifd, exec);
        result += execute("ip addr add %address%/%netmask% dev %iface%", ifd, exec);
-       result += execute("[[ ip route add ::/0 via %gateway% ]]", ifd, exec);
+       result += execute("[[ip route add ::/0 via %gateway%]]", ifd, exec);
        return ((result == 4) ? 4 : 0);
 }
 
@@ -379,7 +371,7 @@ static int v4tunnel_down(struct interface_defn_t * ifd, execfn * exec)
 #endif
 
 static const struct method_t methods6[] = {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        { "v4tunnel", v4tunnel_up, v4tunnel_down, },
 #endif
        { "static", static_up6, static_down6, },
@@ -391,12 +383,12 @@ static const struct address_family_t addr_inet6 = {
        sizeof(methods6) / sizeof(struct method_t),
        methods6
 };
-#endif /* CONFIG_FEATURE_IFUPDOWN_IPV6 */
+#endif /* FEATURE_IFUPDOWN_IPV6 */
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPV4
+#if ENABLE_FEATURE_IFUPDOWN_IPV4
 static int loopback_up(struct interface_defn_t *ifd, execfn *exec)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        int result;
        result = execute("ip addr add 127.0.0.1/8 dev %iface%", ifd, exec);
        result += execute("ip link set %iface% up", ifd, exec);
@@ -408,7 +400,7 @@ static int loopback_up(struct interface_defn_t *ifd, execfn *exec)
 
 static int loopback_down(struct interface_defn_t *ifd, execfn *exec)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        int result;
        result = execute("ip addr flush dev %iface%", ifd, exec);
        result += execute("ip link set %iface% down", ifd, exec);
@@ -421,90 +413,135 @@ static int loopback_down(struct interface_defn_t *ifd, execfn *exec)
 static int static_up(struct interface_defn_t *ifd, execfn *exec)
 {
        int result;
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
-       result = execute("ip addr add %address%/%bnmask% [[broadcast %broadcast%]] "
-                       "dev %iface% [[peer %pointopoint%]] [[label %label%]]", ifd, exec);
-       result += execute("ip link set [[mtu %mtu%]] [[address %hwaddress%]] %iface% up", ifd, exec);
-       result += execute("[[ ip route add default via %gateway% dev %iface% ]]", ifd, exec);
+#if ENABLE_FEATURE_IFUPDOWN_IP
+       result = execute("ip addr add %address%/%bnmask%[[ broadcast %broadcast%]] "
+                       "dev %iface%[[ peer %pointopoint%]][[ label %label%]]", ifd, exec);
+       result += execute("ip link set[[ mtu %mtu%]][[ address %hwaddress%]] %iface% up", ifd, exec);
+       result += execute("[[ip route add default via %gateway% dev %iface%]]", ifd, exec);
        return ((result == 3) ? 3 : 0);
 #else
-       result = execute("ifconfig %iface% %address% netmask %netmask% "
-                       "[[broadcast %broadcast%]] [[pointopoint %pointopoint%]] "
-                       "[[media %media%]] [[mtu %mtu%]] [[hw %hwaddress%]] up",
-                       ifd, exec);
-       result += execute("[[ route add default gw %gateway% %iface% ]]", ifd, exec);
-       return ((result == 2) ? 2 : 0);
+       /* ifconfig said to set iface up before it processes hw %hwaddress%,
+        * which then of course fails. Thus we run two separate ifconfig */
+       result = execute("ifconfig %iface%[[ hw %hwaddress%]][[ media %media%]][[ mtu %mtu%]] up",
+                               ifd, exec);
+       result += execute("ifconfig %iface% %address% netmask %netmask%"
+                               "[[ broadcast %broadcast%]][[ pointopoint %pointopoint%]] ",
+                               ifd, exec);
+       result += execute("[[route add default gw %gateway% %iface%]]", ifd, exec);
+       return ((result == 3) ? 3 : 0);
 #endif
 }
 
 static int static_down(struct interface_defn_t *ifd, execfn *exec)
 {
        int result;
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
+#if ENABLE_FEATURE_IFUPDOWN_IP
        result = execute("ip addr flush dev %iface%", ifd, exec);
        result += execute("ip link set %iface% down", ifd, exec);
 #else
-       result = execute("[[ route del default gw %gateway% %iface% ]]", ifd, exec);
+       result = execute("[[route del default gw %gateway% %iface%]]", ifd, exec);
        result += execute("ifconfig %iface% down", ifd, exec);
 #endif
        return ((result == 2) ? 2 : 0);
 }
 
+#if !ENABLE_APP_UDHCPC
+struct dhcp_client_t
+{
+       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 (execute("udhcpc -n -p /var/run/udhcpc.%iface%.pid -i "
-                       "%iface% [[-H %hostname%]] [[-c %clientid%]]", ifd, exec)) return 1;
-       if (execute("pump -i %iface% [[-h %hostname%]] [[-l %leasehours%]]", ifd, exec)) return 1;
-       if (execute("dhclient -pf /var/run/dhclient.%iface%.pid %iface%", ifd, exec)) return 1;
-       if (execute("dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %clientid%]] "
-                       "[[-l %leasetime%]] %iface%", ifd, exec)) return 1;
+#if ENABLE_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);
+       }
+       bb_error_msg("no dhcp clients found");
        return 0;
+#endif
 }
 
 static int dhcp_down(struct interface_defn_t *ifd, execfn *exec)
 {
-       /* 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 */
-       if (execute("kill -USR2 `cat /var/run/udhcpc.%iface%.pid` 2>/dev/null", ifd, exec)
-        || execute("kill -TERM `cat /var/run/udhcpc.%iface%.pid` 2>/dev/null", ifd, exec))
-               return 1;
-       if (execute("pump -i %iface% -k", ifd, exec)) return 1;
-       if (execute("kill -9 `cat /var/run/dhclient.%iface%.pid` 2>/dev/null", ifd, exec)) return 1;
-       if (execute("dhcpcd -k %iface%", ifd, exec)) return 1;
+#if ENABLE_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);
+       }
+       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)
 {
-       return execute("bootpc [[--bootfile %bootfile%]] --dev %iface% "
-                       "[[--server %server%]] [[--hwaddr %hwaddr%]] "
+       return execute("bootpc[[ --bootfile %bootfile%]] --dev %iface%"
+                       "[[ --server %server%]][[ --hwaddr %hwaddr%]] "
                        "--returniffail --serverbcast", ifd, exec);
 }
 
 static int ppp_up(struct interface_defn_t *ifd, execfn *exec)
 {
-       return execute("pon [[%provider%]]", ifd, exec);
+       return execute("pon[[ %provider%]]", ifd, exec);
 }
 
 static int ppp_down(struct interface_defn_t *ifd, execfn *exec)
 {
-       return execute("poff [[%provider%]]", ifd, exec);
+       return execute("poff[[ %provider%]]", ifd, exec);
 }
 
 static int wvdial_up(struct interface_defn_t *ifd, execfn *exec)
 {
-       return execute("/sbin/start-stop-daemon --start -x /usr/bin/wvdial "
-               "-p /var/run/wvdial.%iface% -b -m -- [[ %provider% ]]", ifd, exec);
+       return execute("start-stop-daemon --start -x wvdial "
+               "-p /var/run/wvdial.%iface% -b -m --[[ %provider%]]", ifd, exec);
 }
 
 static int wvdial_down(struct interface_defn_t *ifd, execfn *exec)
 {
-       return execute("/sbin/start-stop-daemon --stop -x /usr/bin/wvdial "
+       return execute("start-stop-daemon --stop -x wvdial "
                        "-p /var/run/wvdial.%iface% -s 2", ifd, 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, },
@@ -519,22 +556,19 @@ static const struct address_family_t addr_inet = {
        methods
 };
 
-#endif /* ifdef CONFIG_FEATURE_IFUPDOWN_IPV4 */
+#endif /* if ENABLE_FEATURE_IFUPDOWN_IPV4 */
 
 static char *next_word(char **buf)
 {
        unsigned short length;
        char *word;
 
-       if ((buf == NULL) || (*buf == NULL) || (**buf == '\0')) {
+       if (!buf || !*buf || !**buf) {
                return NULL;
        }
 
        /* Skip over leading whitespace */
-       word = *buf;
-       while (isspace(*word)) {
-               ++word;
-       }
+       word = skip_whitespace(*buf);
 
        /* Skip over comments */
        if (*word == '#') {
@@ -560,6 +594,9 @@ static const struct address_family_t *get_address_family(const struct address_fa
 {
        int i;
 
+       if (!name)
+               return NULL;
+
        for (i = 0; af[i]; i++) {
                if (strcmp(af[i]->name, name) == 0) {
                        return af[i];
@@ -572,6 +609,9 @@ static const struct method_t *get_method(const struct address_family_t *af, char
 {
        int i;
 
+       if (!name)
+               return NULL;
+
        for (i = 0; i < af->n_methods; i++) {
                if (strcmp(af->method[i].name, name) == 0) {
                        return &af->method[i];
@@ -582,6 +622,9 @@ static const struct method_t *get_method(const struct address_family_t *af, char
 
 static const llist_t *find_list_string(const llist_t *list, const char *string)
 {
+       if (string == NULL)
+               return NULL;
+
        while (list) {
                if (strcmp(list->data, string) == 0) {
                        return list;
@@ -593,7 +636,7 @@ static const llist_t *find_list_string(const llist_t *list, const char *string)
 
 static struct interfaces_file_t *read_interfaces(const char *filename)
 {
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
        struct mapping_defn_t *currmap = NULL;
 #endif
        struct interface_defn_t *currif = NULL;
@@ -608,7 +651,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);
@@ -618,7 +661,7 @@ static struct interfaces_file_t *read_interfaces(const char *filename)
                }
 
                if (strcmp(firstword, "mapping") == 0) {
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
                        currmap = xzalloc(sizeof(struct mapping_defn_t));
 
                        while ((firstword = next_word(&buf_ptr)) != NULL) {
@@ -646,10 +689,10 @@ static struct interfaces_file_t *read_interfaces(const char *filename)
                        currently_processing = MAPPING;
                } else if (strcmp(firstword, "iface") == 0) {
                        static const struct address_family_t *const addr_fams[] = {
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPV4
+#if ENABLE_FEATURE_IFUPDOWN_IPV4
                                &addr_inet,
 #endif
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPV6
+#if ENABLE_FEATURE_IFUPDOWN_IPV6
                                &addr_inet6,
 #endif
                                NULL
@@ -671,9 +714,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);
@@ -764,7 +805,7 @@ static struct interfaces_file_t *read_interfaces(const char *filename)
                                currif->n_options++;
                                break;
                        case MAPPING:
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
                                if (strcmp(firstword, "script") == 0) {
                                        if (currmap->script != NULL) {
                                                bb_error_msg("duplicate script in mapping \"%s\"", buf);
@@ -832,15 +873,15 @@ static void set_environ(struct interface_defn_t *iface, const char *mode)
        const int n_env_entries = iface->n_options + 5;
        char **ppch;
 
-       if (__myenviron != NULL) {
-               for (ppch = __myenviron; *ppch; ppch++) {
+       if (my_environ != NULL) {
+               for (ppch = my_environ; *ppch; ppch++) {
                        free(*ppch);
                        *ppch = NULL;
                }
-               free(__myenviron);
+               free(my_environ);
        }
-       __myenviron = xzalloc(sizeof(char *) * (n_env_entries + 1 /* for final NULL */ ));
-       environend = __myenviron;
+       my_environ = xzalloc(sizeof(char *) * (n_env_entries + 1 /* for final NULL */ ));
+       environend = my_environ;
 
        for (i = 0; i < iface->n_options; i++) {
                if (strcmp(iface->option[i].name, "up") == 0
@@ -861,19 +902,20 @@ static void set_environ(struct interface_defn_t *iface, const char *mode)
 
 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;
 
                fflush(NULL);
-               switch (child = fork()) {
-               case -1:                /* failure */
+               child = fork();
+               switch (child) {
+               case -1: /* failure */
                        return 0;
-               case 0:         /* child */
-                       execle(DEFAULT_SHELL, DEFAULT_SHELL, "-c", str, NULL, __myenviron);
+               case 0: /* child */
+                       execle(DEFAULT_SHELL, DEFAULT_SHELL, "-c", str, NULL, my_environ);
                        exit(127);
                }
                waitpid(child, &status, 0);
@@ -897,19 +939,18 @@ static int execute_all(struct interface_defn_t *ifd, const char *opt)
        }
 
        buf = xasprintf("run-parts /etc/network/if-%s.d", opt);
-       if (doit(buf) != 1) {
-               return 0;
-       }
-       return 1;
+       /* heh, we don't bother free'ing it */
+       return doit(buf);
 }
 
-static int check(char *str) {
+static int check(char *str)
+{
        return str != NULL;
 }
 
 static int iface_up(struct interface_defn_t *iface)
 {
-       if (!iface->method->up(iface,check)) return -1;
+       if (!iface->method->up(iface, check)) return -1;
        set_environ(iface, "start");
        if (!execute_all(iface, "pre-up")) return 0;
        if (!iface->method->up(iface, doit)) return 0;
@@ -927,7 +968,7 @@ static int iface_down(struct interface_defn_t *iface)
        return 1;
 }
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
 static int popen2(FILE **in, FILE **out, char *command, ...)
 {
        va_list ap;
@@ -1007,7 +1048,7 @@ static char *run_mapping(char *physical, struct mapping_defn_t * map)
                /* If the mapping script exited successfully, try to
                 * grab a line of output and use that as the name of the
                 * logical interface. */
-               char *new_logical = (char *)xmalloc(MAX_INTERFACE_LENGTH);
+               char *new_logical = xmalloc(MAX_INTERFACE_LENGTH);
 
                if (fgets(new_logical, MAX_INTERFACE_LENGTH, out)) {
                        /* If we are able to read a line of output from the script,
@@ -1031,7 +1072,7 @@ static char *run_mapping(char *physical, struct mapping_defn_t * map)
 
        return logical;
 }
-#endif /* CONFIG_FEATURE_IFUPDOWN_MAPPING */
+#endif /* FEATURE_IFUPDOWN_MAPPING */
 
 static llist_t *find_iface_state(llist_t *state_list, const char *iface)
 {
@@ -1050,36 +1091,32 @@ static llist_t *find_iface_state(llist_t *state_list, const char *iface)
 
 int ifupdown_main(int argc, char **argv)
 {
-       static const char statefile[] = "/var/run/ifstate";
-
        int (*cmds)(struct interface_defn_t *) = NULL;
        struct interfaces_file_t *defn;
        llist_t *state_list = NULL;
        llist_t *target_list = NULL;
        const char *interfaces = "/etc/network/interfaces";
        int any_failures = 0;
-       int i;
 
-       if (bb_applet_name[2] == 'u') {
+       cmds = iface_down;
+       if (applet_name[2] == 'u') {
                /* ifup command */
                cmds = iface_up;
-       } else {
-               /* ifdown command */
-               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
+       } else {
                if (!DO_ALL) bb_show_usage();
+       }
 
        debug_noise("reading %s file:\n", interfaces);
        defn = read_interfaces(interfaces);
        debug_noise("\ndone reading %s\n\n", interfaces);
 
        if (!defn) {
-               exit(EXIT_FAILURE);
+               return EXIT_FAILURE;
        }
 
        startup_PATH = getenv("PATH");
@@ -1102,7 +1139,6 @@ int ifupdown_main(int argc, char **argv)
                llist_add_to_end(&target_list, argv[optind]);
        }
 
-
        /* Update the interfaces */
        while (target_list) {
                llist_t *iface_list;
@@ -1142,11 +1178,12 @@ int ifupdown_main(int argc, char **argv)
                        }
                }
 
-#ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
+#if ENABLE_FEATURE_IFUPDOWN_MAPPING
                if ((cmds == iface_up) && !NO_MAPPINGS) {
                        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;
@@ -1160,7 +1197,6 @@ int ifupdown_main(int argc, char **argv)
                }
 #endif
 
-
                iface_list = defn->ifaces;
                while (iface_list) {
                        currif = (struct interface_defn_t *) iface_list->data;
@@ -1177,9 +1213,9 @@ int ifupdown_main(int argc, char **argv)
                                if (cmds_ret == -1) {
                                        bb_error_msg("don't seem to have all the variables for %s/%s",
                                                        liface, currif->address_family->name);
-                                       any_failures += 1;
+                                       any_failures = 1;
                                } else if (cmds_ret == 0) {
-                                       any_failures += 1;
+                                       any_failures = 1;
                                }
 
                                currif->iface = oldiface;
@@ -1192,7 +1228,7 @@ int ifupdown_main(int argc, char **argv)
 
                if (!okay && !FORCE) {
                        bb_error_msg("ignoring unknown interface %s", liface);
-                       any_failures += 1;
+                       any_failures = 1;
                } else {
                        llist_t *iface_state = find_iface_state(state_list, iface);
 
@@ -1213,20 +1249,17 @@ int ifupdown_main(int argc, char **argv)
 
        /* Actually write the new state */
        if (!NO_ACT) {
-               FILE *state_fp = NULL;
+               FILE *state_fp;
 
-               state_fp = xfopen(statefile, "w");
+               state_fp = xfopen("/var/run/ifstate", "w");
                while (state_list) {
                        if (state_list->data) {
-                               fputs(state_list->data, state_fp);
-                               fputc('\n', state_fp);
+                               fprintf(state_fp, "%s\n", state_list->data);
                        }
                        state_list = state_list->link;
                }
                fclose(state_fp);
        }
 
-       if (any_failures)
-               return 1;
-       return 0;
+       return any_failures;
 }