- spelling
[oweals/busybox.git] / networking / ifupdown.c
index 276ca5f22426b17c5f99545754532d0fc51d8ff2..6a200bb2668c1b53598c269ab4e15c34a396bc72 100644 (file)
  *  Lines in the interfaces file cannot wrap.
  *  To adhere to the FHS, the default state file is /var/run/ifstate.
  *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, write to the Free Software
- *  Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * Licensed under the GPL v2 or later, see the file LICENSE in this tarball.
  */
 
 /* TODO: standardise execute() return codes to return 0 for success and 1 for failure */
@@ -44,7 +32,7 @@
 #include <string.h>
 #include <unistd.h>
 
-#include "libbb.h"
+#include "busybox.h"
 
 #define MAX_OPT_DEPTH 10
 #define EUNBALBRACK 10001
 struct interface_defn_t;
 
 typedef int (execfn)(char *command);
-typedef int (command_set)(struct interface_defn_t *ifd, execfn *e);
-
-extern llist_t *llist_add_to_end(llist_t *list_head, char *data)
-{
-       llist_t *new_item, *tmp, *prev;
-
-       new_item = xmalloc(sizeof(llist_t));
-       new_item->data = data;
-       new_item->link = NULL;
-
-       prev = NULL;
-       tmp = list_head;
-       while(tmp) {
-               prev = tmp;
-               tmp = tmp->link;
-       }
-       if (prev) {
-               prev->link = new_item;
-       } else {
-               list_head = new_item;
-       }
-
-       return(list_head);
-}
 
 struct method_t
 {
        char *name;
-       command_set *up;
-       command_set *down;
+       int (*up)(struct interface_defn_t *ifd, execfn *e);
+       int (*down)(struct interface_defn_t *ifd, execfn *e);
 };
 
 struct address_family_t
@@ -127,15 +91,10 @@ struct variable_t
 
 struct interface_defn_t
 {
-       struct interface_defn_t *prev;
-       struct interface_defn_t *next;
-
-       char *iface;
        struct address_family_t *address_family;
        struct method_t *method;
 
-       int automatic;
-
+       char *iface;
        int max_options;
        int n_options;
        struct variable_t *option;
@@ -150,7 +109,7 @@ struct interfaces_file_t
 
 static char no_act = 0;
 static char verbose = 0;
-static char **environ = NULL;
+static char **__myenviron = NULL;
 
 #ifdef CONFIG_FEATURE_IFUPDOWN_IP
 
@@ -176,6 +135,7 @@ static int count_netmask_bits(char *dotted_quad)
 }
 #endif
 
+#if ENABLE_FEATURE_IFUPDOWN_IPV4 || ENABLE_FEATURE_IFUPDOWN_IPV6
 static void addstr(char **buf, size_t *len, size_t *pos, char *str, size_t str_length)
 {
        if (*pos + str_length >= *len) {
@@ -296,7 +256,7 @@ static char *parse(char *command, struct interface_defn_t *ifd)
                                        varvalue = get_var(command, nextpercent - command, ifd);
 
                                        if (varvalue) {
-                                               addstr(&result, &len, &pos, varvalue, bb_strlen(varvalue));
+                                               addstr(&result, &len, &pos, varvalue, strlen(varvalue));
                                        } else {
 #ifdef CONFIG_FEATURE_IFUPDOWN_IP
                                                /* Sigh...  Add a special case for 'ip' to convert from
@@ -307,7 +267,7 @@ static char *parse(char *command, struct interface_defn_t *ifd)
                                                        if (varvalue && (res=count_netmask_bits(varvalue)) > 0) {
                                                                char argument[255];
                                                                sprintf(argument, "%d", res);
-                                                               addstr(&result, &len, &pos, argument, bb_strlen(argument));
+                                                               addstr(&result, &len, &pos, argument, strlen(argument));
                                                                command = nextpercent + 1;
                                                                break;
                                                        }
@@ -354,39 +314,7 @@ static int execute(char *command, struct interface_defn_t *ifd, execfn *exec)
        }
        return(1);
 }
-
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPX
-static int static_up_ipx(struct interface_defn_t *ifd, execfn *exec)
-{
-       return(execute("ipx_interface add %iface% %frame% %netnum%", ifd, exec));
-}
-
-static int static_down_ipx(struct interface_defn_t *ifd, execfn *exec)
-{
-       return(execute("ipx_interface del %iface% %frame%", ifd, exec));
-}
-
-static int dynamic_up(struct interface_defn_t *ifd, execfn *exec)
-{
-       return(execute("ipx_interface add %iface% %frame%", ifd, exec));
-}
-
-static int dynamic_down(struct interface_defn_t *ifd, execfn *exec)
-{
-       return(execute("ipx_interface del %iface% %frame%", ifd, exec));
-}
-
-static struct method_t methods_ipx[] = {
-       { "dynamic", dynamic_up, dynamic_down, },
-       { "static", static_up_ipx, static_down_ipx, },
-};
-
-struct address_family_t addr_ipx = {
-       "ipx",
-       sizeof(methods_ipx) / sizeof(struct method_t),
-       methods_ipx
-};
-#endif /* IFUP_FEATURE_IPX */
+#endif
 
 #ifdef CONFIG_FEATURE_IFUPDOWN_IPV6
 static int loopback_up6(struct interface_defn_t *ifd, execfn *exec)
@@ -460,7 +388,7 @@ static struct method_t methods6[] = {
        { "loopback", loopback_up6, loopback_down6, },
 };
 
-struct address_family_t addr_inet6 = {
+static struct address_family_t addr_inet6 = {
        "inet6",
        sizeof(methods6) / sizeof(struct method_t),
        methods6
@@ -503,8 +431,8 @@ static int static_up(struct interface_defn_t *ifd, execfn *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",
+                               "[[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);
@@ -551,21 +479,15 @@ static int dhcp_up(struct interface_defn_t *ifd, execfn *exec)
        return(0);
 }
 
-static int bootp_down(struct interface_defn_t *ifd, execfn *exec)
-{
-#ifdef CONFIG_FEATURE_IFUPDOWN_IP
-       return(execute("ip link set %iface% down", ifd, exec));
-#else
-       return(execute("ifconfig %iface% down", ifd, exec));
-#endif
-}
-
 static int dhcp_down(struct interface_defn_t *ifd, execfn *exec)
 {
        int result = 0;
        if (execable("/sbin/udhcpc")) {
-               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);
+               /* 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")) {
@@ -573,7 +495,7 @@ static int dhcp_down(struct interface_defn_t *ifd, execfn *exec)
        } else if (execable("/sbin/dhcpcd")) {
                result = execute("dhcpcd -k %iface%", ifd, exec);
        }
-       return (result || bootp_down(ifd, exec));
+       return (result || static_down(ifd, exec));
 }
 
 static int bootp_up(struct interface_defn_t *ifd, execfn *exec)
@@ -610,12 +532,12 @@ static struct method_t methods[] =
        { "wvdial", wvdial_up, wvdial_down, },
        { "ppp", ppp_up, ppp_down, },
        { "static", static_up, static_down, },
-       { "bootp", bootp_up, bootp_down, },
+       { "bootp", bootp_up, static_down, },
        { "dhcp", dhcp_up, dhcp_down, },
        { "loopback", loopback_up, loopback_down, },
 };
 
-struct address_family_t addr_inet =
+static struct address_family_t addr_inet =
 {
        "inet",
        sizeof(methods) / sizeof(struct method_t),
@@ -683,17 +605,6 @@ static struct method_t *get_method(struct address_family_t *af, char *name)
        return(NULL);
 }
 
-static int duplicate_if(struct interface_defn_t *ifa, struct interface_defn_t *ifb)
-{
-       if (strcmp(ifa->iface, ifb->iface) != 0) {
-               return(0);
-       }
-       if (ifa->address_family != ifb->address_family) {
-               return(0);
-       }
-       return(1);
-}
-
 static const llist_t *find_list_string(const llist_t *list, const char *string)
 {
        while (list) {
@@ -705,7 +616,7 @@ static const llist_t *find_list_string(const llist_t *list, const char *string)
        return(NULL);
 }
 
-static struct interfaces_file_t *read_interfaces(char *filename)
+static struct interfaces_file_t *read_interfaces(const char *filename)
 {
 #ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
        struct mapping_defn_t *currmap = NULL;
@@ -718,10 +629,7 @@ static struct interfaces_file_t *read_interfaces(char *filename)
 
        enum { NONE, IFACE, MAPPING } currently_processing = NONE;
 
-       defn = xmalloc(sizeof(struct interfaces_file_t));
-       defn->autointerfaces = NULL;
-       defn->mappings = NULL;
-       defn->ifaces = NULL;
+       defn = xzalloc(sizeof(struct interfaces_file_t));
 
        f = bb_xfopen(filename, "r");
 
@@ -736,10 +644,7 @@ static struct interfaces_file_t *read_interfaces(char *filename)
 
                if (strcmp(firstword, "mapping") == 0) {
 #ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
-                       currmap = xmalloc(sizeof(struct mapping_defn_t));
-                       currmap->max_matches = 0;
-                       currmap->n_matches = 0;
-                       currmap->match = NULL;
+                       currmap = xzalloc(sizeof(struct mapping_defn_t));
 
                        while ((firstword = next_word(&buf_ptr)) != NULL) {
                                if (currmap->max_matches == currmap->n_matches) {
@@ -775,14 +680,11 @@ static struct interfaces_file_t *read_interfaces(char *filename)
 #endif
 #ifdef CONFIG_FEATURE_IFUPDOWN_IPV6
                                        &addr_inet6,
-#endif
-#ifdef CONFIG_FEATURE_IFUPDOWN_IPX
-                                       &addr_ipx,
 #endif
                                        NULL
                                };
 
-                               currif = xmalloc(sizeof(struct interface_defn_t));
+                               currif = xzalloc(sizeof(struct interface_defn_t));
                                iface_name = next_word(&buf_ptr);
                                address_family_name = next_word(&buf_ptr);
                                method_name = next_word(&buf_ptr);
@@ -816,25 +718,19 @@ static struct interfaces_file_t *read_interfaces(char *filename)
                                        return NULL;
                                }
 
-                               currif->automatic = 1;
-                               currif->max_options = 0;
-                               currif->n_options = 0;
-                               currif->option = NULL;
 
                                {
-                                       struct interface_defn_t *tmp;
                                        llist_t *iface_list;
-                                       iface_list = defn->ifaces;
-                                       while (iface_list) {
-                                               tmp = (struct interface_defn_t *) iface_list->data;
-                                               if (duplicate_if(tmp, currif)) {
+                                       for (iface_list = defn->ifaces; iface_list; iface_list = iface_list->link) {
+                                               struct interface_defn_t *tmp = (struct interface_defn_t *) iface_list->data;
+                                               if ((strcmp(tmp->iface, currif->iface) == 0) && 
+                                                       (tmp->address_family == currif->address_family)) {
                                                        bb_error_msg("duplicate interface \"%s\"", tmp->iface);
                                                        return NULL;
                                                }
-                                               iface_list = iface_list->link;
                                        }
 
-                                       defn->ifaces = llist_add_to_end(defn->ifaces, (char*)currif);
+                                       llist_add_to_end(&(defn->ifaces), (char*)currif);
                                }
                                debug_noise("iface %s %s %s\n", currif->iface, address_family_name, method_name);
                        }
@@ -848,7 +744,7 @@ static struct interfaces_file_t *read_interfaces(char *filename)
                                }
 
                                /* Add the interface to the list */
-                               defn->autointerfaces = llist_add_to_end(defn->autointerfaces, strdup(firstword));
+                               llist_add_to_end(&(defn->autointerfaces), bb_xstrdup(firstword));
                                debug_noise("\nauto %s\n", firstword);
                        }
                        currently_processing = NONE;
@@ -858,7 +754,7 @@ static struct interfaces_file_t *read_interfaces(char *filename)
                                        {
                                                int i;
 
-                                               if (bb_strlen(buf_ptr) == 0) {
+                                               if (strlen(buf_ptr) == 0) {
                                                        bb_error_msg("option with empty value \"%s\"", buf);
                                                        return NULL;
                                                }
@@ -934,15 +830,13 @@ static struct interfaces_file_t *read_interfaces(char *filename)
        return defn;
 }
 
-static char *setlocalenv(char *format, char *name, char *value)
+static char *setlocalenv(char *format, const char *name, const char *value)
 {
        char *result;
        char *here;
        char *there;
 
-       result = xmalloc(bb_strlen(format) + bb_strlen(name) + bb_strlen(value) + 1);
-
-       sprintf(result, format, name, value);
+       result = bb_xasprintf(format, name, value);
 
        for (here = there = result; *there != '=' && *there; there++) {
                if (*there == '-')
@@ -955,29 +849,27 @@ static char *setlocalenv(char *format, char *name, char *value)
                        here++;
                }
        }
-       memmove(here, there, bb_strlen(there) + 1);
+       memmove(here, there, strlen(there) + 1);
 
        return result;
 }
 
-static void set_environ(struct interface_defn_t *iface, char *mode)
+static void set_environ(struct interface_defn_t *iface, const char *mode)
 {
        char **environend;
        int i;
        const int n_env_entries = iface->n_options + 5;
        char **ppch;
 
-       if (environ != NULL) {
-               for (ppch = environ; *ppch; ppch++) {
+       if (__myenviron != NULL) {
+               for (ppch = __myenviron; *ppch; ppch++) {
                        free(*ppch);
                        *ppch = NULL;
                }
-               free(environ);
-               environ = NULL;
+               free(__myenviron);
        }
-       environ = xmalloc(sizeof(char *) * (n_env_entries + 1 /* for final NULL */ ));
-       environend = environ;
-       *environend = NULL;
+       __myenviron = xzalloc(sizeof(char *) * (n_env_entries + 1 /* for final NULL */ ));
+       environend = __myenviron;
 
        for (i = 0; i < iface->n_options; i++) {
                if (strcmp(iface->option[i].name, "up") == 0
@@ -987,19 +879,13 @@ static void set_environ(struct interface_defn_t *iface, char *mode)
                        continue;
                }
                *(environend++) = setlocalenv("IF_%s=%s", iface->option[i].name, iface->option[i].value);
-               *environend = NULL;
        }
 
        *(environend++) = setlocalenv("%s=%s", "IFACE", iface->iface);
-       *environend = NULL;
        *(environend++) = setlocalenv("%s=%s", "ADDRFAM", iface->address_family->name);
-       *environend = NULL;
        *(environend++) = setlocalenv("%s=%s", "METHOD", iface->method->name);
-       *environend = NULL;
        *(environend++) = setlocalenv("%s=%s", "MODE", mode);
-       *environend = NULL;
        *(environend++) = setlocalenv("%s=%s", "PATH", "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin");
-       *environend = NULL;
 }
 
 static int doit(char *str)
@@ -1016,7 +902,7 @@ static int doit(char *str)
                        case -1:                /* failure */
                                return 0;
                        case 0:         /* child */
-                               execle(DEFAULT_SHELL, DEFAULT_SHELL, "-c", str, NULL, environ);
+                               execle(DEFAULT_SHELL, DEFAULT_SHELL, "-c", str, NULL, __myenviron);
                                exit(127);
                }
                waitpid(child, &status, 0);
@@ -1027,20 +913,20 @@ static int doit(char *str)
        return (1);
 }
 
-static int execute_all(struct interface_defn_t *ifd, execfn *exec, const char *opt)
+static int execute_all(struct interface_defn_t *ifd, const char *opt)
 {
        int i;
        char *buf;
        for (i = 0; i < ifd->n_options; i++) {
                if (strcmp(ifd->option[i].name, opt) == 0) {
-                       if (!(*exec) (ifd->option[i].value)) {
+                       if (!doit(ifd->option[i].value)) {
                                return 0;
                        }
                }
        }
 
-       bb_xasprintf(&buf, "run-parts /etc/network/if-%s.d", opt);
-       if ((*exec)(buf) != 1) {
+       buf = bb_xasprintf("run-parts /etc/network/if-%s.d", opt);
+       if (doit(buf) != 1) {
                return 0;
        }
        return 1;
@@ -1054,9 +940,9 @@ static int iface_up(struct interface_defn_t *iface)
 {
        if (!iface->method->up(iface,check)) return -1;
        set_environ(iface, "start");
-       if (!execute_all(iface, doit, "pre-up")) return 0;
+       if (!execute_all(iface, "pre-up")) return 0;
        if (!iface->method->up(iface, doit)) return 0;
-       if (!execute_all(iface, doit, "up")) return 0;
+       if (!execute_all(iface, "up")) return 0;
        return 1;
 }
 
@@ -1064,9 +950,9 @@ static int iface_down(struct interface_defn_t *iface)
 {
        if (!iface->method->down(iface,check)) return -1;
        set_environ(iface, "stop");
-       if (!execute_all(iface, doit, "down")) return 0;
+       if (!execute_all(iface, "down")) return 0;
        if (!iface->method->down(iface, doit)) return 0;
-       if (!execute_all(iface, doit, "post-down")) return 0;
+       if (!execute_all(iface, "post-down")) return 0;
        return 1;
 }
 
@@ -1156,7 +1042,7 @@ static char *run_mapping(char *physical, struct mapping_defn_t * map)
                        /* If we are able to read a line of output from the script,
                         * remove any trailing whitespace and use this value
                         * as the name of the logical interface. */
-                       char *pch = new_logical + bb_strlen(new_logical) - 1;
+                       char *pch = new_logical + strlen(new_logical) - 1;
 
                        while (pch >= new_logical && isspace(*pch))
                                *(pch--) = '\0';
@@ -1164,7 +1050,7 @@ static char *run_mapping(char *physical, struct mapping_defn_t * map)
                        free(logical);
                        logical = new_logical;
                } else {
-                       /* If we are UNABLE to read a line of output, discard are
+                       /* If we are UNABLE to read a line of output, discard our
                         * freshly allocated memory. */
                        free(new_logical);
                }
@@ -1178,7 +1064,7 @@ static char *run_mapping(char *physical, struct mapping_defn_t * map)
 
 static llist_t *find_iface_state(llist_t *state_list, const char *iface)
 {
-       unsigned short iface_len = bb_strlen(iface);
+       unsigned short iface_len = strlen(iface);
        llist_t *search = state_list;
 
        while (search) {
@@ -1191,14 +1077,13 @@ static llist_t *find_iface_state(llist_t *state_list, const char *iface)
        return(NULL);
 }
 
-extern int ifupdown_main(int argc, char **argv)
+int ifupdown_main(int argc, char **argv)
 {
        int (*cmds) (struct interface_defn_t *) = NULL;
        struct interfaces_file_t *defn;
-       FILE *state_fp = NULL;
        llist_t *state_list = NULL;
        llist_t *target_list = NULL;
-       char *interfaces = "/etc/network/interfaces";
+       const char *interfaces = "/etc/network/interfaces";
        const char *statefile = "/var/run/ifstate";
 
 #ifdef CONFIG_FEATURE_IFUPDOWN_MAPPING
@@ -1225,7 +1110,7 @@ extern int ifupdown_main(int argc, char **argv)
                {
                        switch (i) {
                                case 'i':       /* interfaces */
-                                       interfaces = bb_xstrdup(optarg);
+                                       interfaces = optarg;
                                        break;
                                case 'v':       /* verbose */
                                        verbose = 1;
@@ -1268,48 +1153,21 @@ extern int ifupdown_main(int argc, char **argv)
                exit(EXIT_FAILURE);
        }
 
-       if (no_act) {
-               state_fp = fopen(statefile, "r");
-       }
-
        /* Create a list of interfaces to work on */
        if (do_all) {
                if (cmds == iface_up) {
                        target_list = defn->autointerfaces;
                } else {
-#if 0
-                       /* iface_down */
-                       llist_t *new_item;
-                       const llist_t *list = state_list;
-                       while (list) {
-                               new_item = xmalloc(sizeof(llist_t));
-                               new_item->data = strdup(list->data);
-                               new_item->link = NULL;
-                               list = target_list;
-                               if (list == NULL)
-                                       target_list = new_item;
-                               else {
-                                       while (list->link) {
-                                               list = list->link;
-                                       }
-                                       list = new_item;
-                               }
-                               list = list->link;
-                       }
-                       target_list = defn->autointerfaces;
-#else
-
                        /* iface_down */
                        const llist_t *list = state_list;
                        while (list) {
-                               target_list = llist_add_to_end(target_list, strdup(list->data));
+                               llist_add_to_end(&target_list, bb_xstrdup(list->data));
                                list = list->link;
                        }
                        target_list = defn->autointerfaces;
-#endif
                }
        } else {
-               target_list = llist_add_to_end(target_list, argv[optind]);
+               llist_add_to_end(&target_list, argv[optind]);
        }
 
 
@@ -1323,15 +1181,15 @@ extern int ifupdown_main(int argc, char **argv)
                int okay = 0;
                int cmds_ret;
 
-               iface = strdup(target_list->data);
+               iface = bb_xstrdup(target_list->data);
                target_list = target_list->link;
 
                pch = strchr(iface, '=');
                if (pch) {
                        *pch = '\0';
-                       liface = strdup(pch + 1);
+                       liface = bb_xstrdup(pch + 1);
                } else {
-                       liface = strdup(iface);
+                       liface = bb_xstrdup(iface);
                }
 
                if (!force) {
@@ -1408,45 +1266,25 @@ extern int ifupdown_main(int argc, char **argv)
                        llist_t *iface_state = find_iface_state(state_list, iface);
 
                        if (cmds == iface_up) {
-                               char *newiface = xmalloc(bb_strlen(iface) + 1 + bb_strlen(liface) + 1);
-                               sprintf(newiface, "%s=%s", iface, liface);
+                               char *newiface = bb_xasprintf("%s=%s", iface, liface);
                                if (iface_state == NULL) {
-                                       state_list = llist_add_to_end(state_list, newiface);
+                                       llist_add_to_end(&state_list, newiface);
                                } else {
                                        free(iface_state->data);
                                        iface_state->data = newiface;
                                }
-                       } else if (cmds == iface_down) {
+                       } else {
                                /* Remove an interface from the linked list */
-                               if (iface_state) {
-                                       /* This needs to be done better */
-                                       free(iface_state->data);
-                                       free(iface_state->link);
-                                       if (iface_state->link) {
-                                               iface_state->data = iface_state->link->data;
-                                               iface_state->link = iface_state->link->link;
-                                       } else {
-                                               iface_state->data = NULL;
-                                               iface_state->link = NULL;
-                                       }
-                               }
+                               free(llist_pop(&iface_state));
                        }
                }
        }
 
        /* Actually write the new state */
        if (!no_act) {
+               FILE *state_fp = NULL;
 
-               if (state_fp)
-                       fclose(state_fp);
-               state_fp = bb_xfopen(statefile, "a+");
-
-               if (ftruncate(fileno(state_fp), 0) < 0) {
-                       bb_error_msg_and_die("failed to truncate statefile %s: %s", statefile, strerror(errno));
-               }
-
-               rewind(state_fp);
-
+               state_fp = bb_xfopen(statefile, "w");
                while (state_list) {
                        if (state_list->data) {
                                fputs(state_list->data, state_fp);
@@ -1454,13 +1292,7 @@ extern int ifupdown_main(int argc, char **argv)
                        }
                        state_list = state_list->link;
                }
-               fflush(state_fp);
-       }
-
-       /* Cleanup */
-       if (state_fp != NULL) {
                fclose(state_fp);
-               state_fp = NULL;
        }
 
        if (any_failures)