Also read addresses from "ipv6-prefix-assignment" ifstatus table
[project/firewall3.git] / main.c
diff --git a/main.c b/main.c
index 90ef820..d4d7b7b 100644 (file)
--- a/main.c
+++ b/main.c
@@ -172,7 +172,7 @@ stop(struct fw3_state *state, bool complete, bool reload)
        }
 
        if (!print_rules)
-               fw3_hotplug_zones(false, state);
+               fw3_hotplug_zones(state, false);
 
        for (family = FW3_FAMILY_V4; family <= FW3_FAMILY_V6; family++)
        {
@@ -199,12 +199,12 @@ stop(struct fw3_state *state, bool complete, bool reload)
                        else
                        {
                                /* pass 1 */
-                               fw3_flush_rules(table, family, false, reload, state);
-                               fw3_flush_zones(table, family, false, reload, state);
+                               fw3_flush_rules(state, family, table, reload, false);
+                               fw3_flush_zones(state, family, table, reload, false);
 
                                /* pass 2 */
-                               fw3_flush_rules(table, family, true, reload, state);
-                               fw3_flush_zones(table, family, true, reload, state);
+                               fw3_flush_rules(state, family, table, reload, true);
+                               fw3_flush_zones(state, family, table, reload, true);
                        }
 
                        fw3_pr("COMMIT\n");
@@ -282,14 +282,14 @@ start(struct fw3_state *state, bool reload)
                             fw3_flag_names[family], fw3_flag_names[table]);
 
                        fw3_pr("*%s\n", fw3_flag_names[table]);
-                       fw3_print_default_chains(table, family, reload, state);
-                       fw3_print_zone_chains(table, family, reload, state);
-                       fw3_print_default_head_rules(table, family, reload, state);
-                       fw3_print_rules(table, family, state);
-                       fw3_print_redirects(table, family, state);
-                       fw3_print_forwards(table, family, state);
-                       fw3_print_zone_rules(table, family, reload, state);
-                       fw3_print_default_tail_rules(table, family, reload, state);
+                       fw3_print_default_chains(state, family, table, reload);
+                       fw3_print_zone_chains(state, family, table, reload);
+                       fw3_print_default_head_rules(state, family, table, reload);
+                       fw3_print_rules(state, family, table);
+                       fw3_print_redirects(state, family, table);
+                       fw3_print_forwards(state, family, table);
+                       fw3_print_zone_rules(state, family, table, reload);
+                       fw3_print_default_tail_rules(state, family, table, reload);
                        fw3_pr("COMMIT\n");
                }
 
@@ -308,7 +308,7 @@ start(struct fw3_state *state, bool reload)
                if (!print_rules)
                {
                        fw3_run_includes(state, reload);
-                       fw3_hotplug_zones(true, state);
+                       fw3_hotplug_zones(state, true);
                        fw3_write_statefile(state);
                }
        }
@@ -404,9 +404,6 @@ int main(int argc, char **argv)
        state = build_state();
        defs = &state->defaults;
 
-       if (!fw3_lock())
-               goto out;
-
        if (optind >= argc)
        {
                rv = usage();
@@ -430,28 +427,50 @@ int main(int argc, char **argv)
        }
        else if (!strcmp(argv[optind], "start"))
        {
-               rv = start(state, false);
+               if (fw3_lock())
+               {
+                       rv = start(state, false);
+                       fw3_unlock();
+               }
        }
        else if (!strcmp(argv[optind], "stop"))
        {
-               rv = stop(state, false, false);
+               if (fw3_lock())
+               {
+                       rv = stop(state, false, false);
+                       fw3_unlock();
+               }
        }
        else if (!strcmp(argv[optind], "flush"))
        {
-               rv = stop(state, true, false);
+               if (fw3_lock())
+               {
+                       rv = stop(state, true, false);
+                       fw3_unlock();
+               }
        }
        else if (!strcmp(argv[optind], "restart"))
        {
-               stop(state, true, false);
-               free_state(state);
+               if (fw3_lock())
+               {
+                       stop(state, true, false);
+                       free_state(state);
 
-               state = build_state();
-               rv = start(state, false);
+                       state = build_state();
+                       rv = start(state, false);
+
+                       fw3_unlock();
+               }
        }
        else if (!strcmp(argv[optind], "reload"))
        {
-               rv = stop(state, false, true);
-               rv = start(state, !rv);
+               if (fw3_lock())
+               {
+                       rv = stop(state, false, true);
+                       rv = start(state, !rv);
+
+                       fw3_unlock();
+               }
        }
        else if (!strcmp(argv[optind], "network") && (optind + 1) < argc)
        {
@@ -470,7 +489,5 @@ out:
        if (state)
                free_state(state);
 
-       fw3_unlock();
-
        return rv;
 }