X-Git-Url: http://git.archive.openwrt.org/?p=project%2Ffirewall3.git;a=blobdiff_plain;f=main.c;h=5580c7d144a38dd605ab5499b1c6e54c900ee35c;hp=3654c1503842929c50cafe8f84f97f404176c49a;hb=b6bfb7a067c5699fd5aec28d407d773d3a022567;hpb=24e1ba98e43d44ad7ba5780afd0b98e19ee14570

diff --git a/main.c b/main.c
index 3654c15..5580c7d 100644
--- a/main.c
+++ b/main.c
@@ -31,17 +31,17 @@
 
 
 static bool print_rules = false;
-static enum fw3_family use_family = FW3_FAMILY_ANY;
 
+static struct fw3_state *run_state = NULL;
+static struct fw3_state *cfg_state = NULL;
 
-static struct fw3_state *
-build_state(void)
+
+static bool
+build_state(bool runtime)
 {
 	struct fw3_state *state = NULL;
 	struct uci_package *p = NULL;
-
-	if (!fw3_ubus_connect())
-		error("Failed to connect to ubus");
+	FILE *sf;
 
 	state = malloc(sizeof(*state));
 
@@ -54,20 +54,47 @@ build_state(void)
 	if (!state->uci)
 		error("Out of memory");
 
-	if (uci_load(state->uci, "firewall", &p))
+	if (runtime)
 	{
-		uci_perror(state->uci, NULL);
-		error("Failed to load /etc/config/firewall");
-	}
+		sf = fopen(FW3_STATEFILE, "r");
 
-	if (!fw3_find_command("ipset"))
-	{
-		warn("Unable to locate ipset utility, disabling ipset support");
-		state->disable_ipsets = true;
+		if (sf)
+		{
+			uci_import(state->uci, sf, "fw3_state", &p, true);
+			fclose(sf);
+		}
+
+		if (!p)
+		{
+			uci_free_context(state->uci);
+			free(state);
+
+			return false;
+		}
+
+		state->statefile = true;
+
+		run_state = state;
 	}
+	else
+	{
+		if (!fw3_ubus_connect())
+			error("Failed to connect to ubus");
 
-	INIT_LIST_HEAD(&state->running_zones);
-	INIT_LIST_HEAD(&state->running_ipsets);
+		if (uci_load(state->uci, "firewall", &p))
+		{
+			uci_perror(state->uci, NULL);
+			error("Failed to load /etc/config/firewall");
+		}
+
+		if (!fw3_find_command("ipset"))
+		{
+			warn("Unable to locate ipset utility, disabling ipset support");
+			state->disable_ipsets = true;
+		}
+
+		cfg_state = state;
+	}
 
 	fw3_load_defaults(state, p);
 	fw3_load_ipsets(state, p);
@@ -77,9 +104,7 @@ build_state(void)
 	fw3_load_forwards(state, p);
 	fw3_load_includes(state, p);
 
-	state->statefile = fw3_read_statefile(state);
-
-	return state;
+	return true;
 }
 
 static void
@@ -133,43 +158,33 @@ restore_pipe(enum fw3_family family, bool silent)
 }
 
 static bool
-family_running(struct fw3_state *state, enum fw3_family family)
-{
-	return hasbit(state->defaults.running_flags, family);
-}
-
-static bool
-family_used(enum fw3_family family)
-{
-	return (use_family == FW3_FAMILY_ANY) || (use_family == family);
-}
-
-static bool
-family_loaded(struct fw3_state *state, enum fw3_family family)
+family_running(enum fw3_family family)
 {
-	return hasbit(state->defaults.flags, family);
+	return (run_state && has(run_state->defaults.flags, family, family));
 }
 
 static void
 family_set(struct fw3_state *state, enum fw3_family family, bool set)
 {
+	if (!state)
+		return;
+
 	if (set)
-		setbit(state->defaults.flags, family);
+		set(state->defaults.flags, family, family);
 	else
-		delbit(state->defaults.flags, family);
+		del(state->defaults.flags, family, family);
 }
 
 static int
-stop(struct fw3_state *state, bool complete, bool reload)
+stop(bool complete, bool reload)
 {
 	FILE *ct;
 
 	int rv = 1;
 	enum fw3_family family;
 	enum fw3_table table;
-	enum fw3_target policy = reload ? FW3_TARGET_DROP : FW3_TARGET_ACCEPT;
 
-	if (!complete && !state->statefile)
+	if (!complete && !run_state)
 	{
 		if (!reload)
 			warn("The firewall appears to be stopped. "
@@ -178,23 +193,24 @@ stop(struct fw3_state *state, bool complete, bool reload)
 		return rv;
 	}
 
+	if (!print_rules && run_state)
+		fw3_hotplug_zones(run_state, false);
+
 	for (family = FW3_FAMILY_V4; family <= FW3_FAMILY_V6; family++)
 	{
-		if (!complete && !family_running(state, family))
+		if (!complete && !family_running(family))
 			continue;
 
-		if (!family_used(family) || !restore_pipe(family, true))
+		if (!restore_pipe(family, true))
 			continue;
 
-		info("Removing %s rules ...", fw3_flag_names[family]);
-
 		for (table = FW3_TABLE_FILTER; table <= FW3_TABLE_RAW; table++)
 		{
 			if (!fw3_has_table(family == FW3_FAMILY_V6, fw3_flag_names[table]))
 				continue;
 
-			info(" * %sing %s table",
-			     complete ? "Flush" : "Clear", fw3_flag_names[table]);
+			info(" * %sing %s %s table", complete ? "Flush" : "Clear",
+			     fw3_flag_names[family], fw3_flag_names[table]);
 
 			fw3_pr("*%s\n", fw3_flag_names[table]);
 
@@ -202,15 +218,15 @@ stop(struct fw3_state *state, bool complete, bool reload)
 			{
 				fw3_flush_all(table);
 			}
-			else
+			else if (run_state)
 			{
 				/* pass 1 */
-				fw3_flush_rules(table, family, false, state, policy);
-				fw3_flush_zones(table, family, false, reload, state);
+				fw3_flush_rules(run_state, family, table, reload, false);
+				fw3_flush_zones(run_state, family, table, reload, false);
 
 				/* pass 2 */
-				fw3_flush_rules(table, family, true, state, policy);
-				fw3_flush_zones(table, family, true, reload, state);
+				fw3_flush_rules(run_state, family, table, reload, true);
+				fw3_flush_zones(run_state, family, table, reload, true);
 			}
 
 			fw3_pr("COMMIT\n");
@@ -218,15 +234,16 @@ stop(struct fw3_state *state, bool complete, bool reload)
 
 		fw3_command_close();
 
-		if (!reload)
+		if (!reload && run_state)
 		{
 			if (fw3_command_pipe(false, "ipset", "-exist", "-"))
 			{
-				fw3_destroy_ipsets(state, family);
+				fw3_destroy_ipsets(run_state, family);
 				fw3_command_close();
 			}
 
-			family_set(state, family, false);
+			family_set(run_state, family, false);
+			family_set(cfg_state, family, false);
 		}
 
 		rv = 0;
@@ -234,20 +251,20 @@ stop(struct fw3_state *state, bool complete, bool reload)
 
 	if (complete && (ct = fopen("/proc/net/nf_conntrack", "w")) != NULL)
 	{
-		info("Flushing conntrack table ...");
+		info(" * Flushing conntrack table ...");
 
 		fwrite("f\n", 2, 1, ct);
 		fclose(ct);
 	}
 
-	if (!rv)
-		fw3_write_statefile(state);
+	if (!rv && run_state)
+		fw3_write_statefile(run_state);
 
 	return rv;
 }
 
 static int
-start(struct fw3_state *state, bool reload)
+start(bool reload)
 {
 	int rv = 1;
 	enum fw3_family family;
@@ -257,17 +274,17 @@ start(struct fw3_state *state, bool reload)
 	{
 		if (fw3_command_pipe(false, "ipset", "-exist", "-"))
 		{
-			fw3_create_ipsets(state);
+			fw3_create_ipsets(cfg_state);
 			fw3_command_close();
 		}
 	}
 
 	for (family = FW3_FAMILY_V4; family <= FW3_FAMILY_V6; family++)
 	{
-		if (!family_used(family))
+		if (family == FW3_FAMILY_V6 && cfg_state->defaults.disable_ipv6)
 			continue;
 
-		if (!print_rules && !reload && family_running(state, family))
+		if (!print_rules && !reload && family_running(family))
 		{
 			warn("The %s firewall appears to be started already. "
 			     "If it is indeed empty, remove the %s file and retry.",
@@ -276,60 +293,60 @@ start(struct fw3_state *state, bool reload)
 			continue;
 		}
 
-		if (!family_loaded(state, family) || !restore_pipe(family, false))
+		if (!restore_pipe(family, false))
 			continue;
 
-		info("Constructing %s rules ...", fw3_flag_names[family]);
-
 		for (table = FW3_TABLE_FILTER; table <= FW3_TABLE_RAW; table++)
 		{
 			if (!fw3_has_table(family == FW3_FAMILY_V6, fw3_flag_names[table]))
 				continue;
 
-			info(" * Populating %s table", fw3_flag_names[table]);
+			info(" * Populating %s %s table",
+			     fw3_flag_names[family], fw3_flag_names[table]);
 
 			fw3_pr("*%s\n", fw3_flag_names[table]);
-			fw3_print_default_chains(table, family, state);
-			fw3_print_zone_chains(table, family, state);
-			fw3_print_default_head_rules(table, family, 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, state);
-			fw3_print_default_tail_rules(table, family, state);
+			fw3_print_default_chains(cfg_state, family, table, reload);
+			fw3_print_zone_chains(cfg_state, family, table, reload);
+			fw3_print_default_head_rules(cfg_state, family, table, reload);
+			fw3_print_rules(cfg_state, family, table);
+			fw3_print_redirects(cfg_state, family, table);
+			fw3_print_forwards(cfg_state, family, table);
+			fw3_print_zone_rules(cfg_state, family, table, reload);
+			fw3_print_default_tail_rules(cfg_state, family, table, reload);
 			fw3_pr("COMMIT\n");
 		}
 
-		if (!reload)
-			fw3_print_includes(family, state);
+		fw3_print_includes(cfg_state, family, reload);
 
 		fw3_command_close();
-		family_set(state, family, true);
+		family_set(run_state, family, true);
+		family_set(cfg_state, family, true);
 
 		rv = 0;
 	}
 
 	if (!rv)
 	{
-		fw3_set_defaults(state);
-
-		if (!reload && !print_rules)
-			fw3_run_includes(state);
+		fw3_set_defaults(cfg_state);
 
 		if (!print_rules)
-			fw3_write_statefile(state);
+		{
+			fw3_run_includes(cfg_state, reload);
+			fw3_hotplug_zones(cfg_state, true);
+			fw3_write_statefile(cfg_state);
+		}
 	}
 
 	return rv;
 }
 
 static int
-lookup_network(struct fw3_state *state, const char *net)
+lookup_network(const char *net)
 {
 	struct fw3_zone *z;
 	struct fw3_device *d;
 
-	list_for_each_entry(z, &state->zones, list)
+	list_for_each_entry(z, &cfg_state->zones, list)
 	{
 		list_for_each_entry(d, &z->networks, list)
 		{
@@ -345,12 +362,12 @@ lookup_network(struct fw3_state *state, const char *net)
 }
 
 static int
-lookup_device(struct fw3_state *state, const char *dev)
+lookup_device(const char *dev)
 {
 	struct fw3_zone *z;
 	struct fw3_device *d;
 
-	list_for_each_entry(z, &state->zones, list)
+	list_for_each_entry(z, &cfg_state->zones, list)
 	{
 		list_for_each_entry(d, &z->devices, list)
 		{
@@ -368,7 +385,8 @@ lookup_device(struct fw3_state *state, const char *dev)
 static int
 usage(void)
 {
-	fprintf(stderr, "fw3 [-4] [-6] [-q] {start|stop|flush|reload|restart|print}\n");
+	fprintf(stderr, "fw3 [-4] [-6] [-q] print\n");
+	fprintf(stderr, "fw3 [-q] {start|stop|flush|reload|restart}\n");
 	fprintf(stderr, "fw3 [-q] network {net}\n");
 	fprintf(stderr, "fw3 [-q] device {dev}\n");
 
@@ -379,8 +397,8 @@ usage(void)
 int main(int argc, char **argv)
 {
 	int ch, rv = 1;
-	struct fw3_state *state = NULL;
 	struct fw3_defaults *defs = NULL;
+	enum fw3_family use_family = FW3_FAMILY_ANY;
 
 	while ((ch = getopt(argc, argv, "46dqh")) != -1)
 	{
@@ -408,11 +426,9 @@ int main(int argc, char **argv)
 		}
 	}
 
-	state = build_state();
-	defs = &state->defaults;
-
-	if (!fw3_lock())
-		goto out;
+	build_state(false);
+	build_state(true);
+	defs = &cfg_state->defaults;
 
 	if (optind >= argc)
 	{
@@ -420,53 +436,71 @@ int main(int argc, char **argv)
 		goto out;
 	}
 
-	if (use_family == FW3_FAMILY_V6 && defs->disable_ipv6)
-		warn("IPv6 rules globally disabled in configuration");
-
 	if (!strcmp(argv[optind], "print"))
 	{
 		if (use_family == FW3_FAMILY_ANY)
 			use_family = FW3_FAMILY_V4;
+		else if (use_family == FW3_FAMILY_V6 && defs->disable_ipv6)
+			warn("IPv6 rules globally disabled in configuration");
 
 		freopen("/dev/null", "w", stderr);
 
-		state->disable_ipsets = true;
+		cfg_state->disable_ipsets = true;
 		print_rules = true;
 
-		rv = start(state, false);
+		rv = start(false);
 	}
 	else if (!strcmp(argv[optind], "start"))
 	{
-		rv = start(state, false);
+		if (fw3_lock())
+		{
+			rv = start(false);
+			fw3_unlock();
+		}
 	}
 	else if (!strcmp(argv[optind], "stop"))
 	{
-		rv = stop(state, false, false);
+		if (fw3_lock())
+		{
+			rv = stop(false, false);
+			fw3_unlock();
+		}
 	}
 	else if (!strcmp(argv[optind], "flush"))
 	{
-		rv = stop(state, true, false);
+		if (fw3_lock())
+		{
+			rv = stop(true, false);
+			fw3_unlock();
+		}
 	}
 	else if (!strcmp(argv[optind], "restart"))
 	{
-		stop(state, true, false);
-		free_state(state);
+		if (fw3_lock())
+		{
+			stop(true, false);
+			rv = start(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(false, true);
+			rv = start(!rv);
+
+			fw3_unlock();
+		}
 	}
 	else if (!strcmp(argv[optind], "network") && (optind + 1) < argc)
 	{
-		rv = lookup_network(state, argv[optind + 1]);
+		rv = lookup_network(argv[optind + 1]);
 	}
 	else if (!strcmp(argv[optind], "device") && (optind + 1) < argc)
 	{
-		rv = lookup_device(state, argv[optind + 1]);
+		rv = lookup_device(argv[optind + 1]);
 	}
 	else
 	{
@@ -474,10 +508,11 @@ int main(int argc, char **argv)
 	}
 
 out:
-	if (state)
-		free_state(state);
+	if (cfg_state)
+		free_state(cfg_state);
 
-	fw3_unlock();
+	if (run_state)
+		free_state(run_state);
 
 	return rv;
 }