Comparison

mod_firewall: Split compile function into two parts in order to separate their scopes

comparison

equal deleted inserted replaced

-:368b091e723b
+:11539785cb92
 	local rule = { conditions = {}, actions = {}, deps = {} };
 	table.insert(ruleset[chain], rule);
 	return rule;
 end
-local function compile_firewall_rules(filename)
+local function parse_firewall_rules(filename)
 	local line_no = 0;
 	local function errmsg(err)
 		return "Error compiling "..filename.." on line "..line_no..": "..err;
 	end
 			for _, dep in ipairs(condition_deps or {}) do
 				table.insert(rule.deps, dep);
 			end
 		end
 	end
+	return ruleset;
+end
+local function process_firewall_rules(ruleset)
 	-- Compile ruleset and return complete code
 	local chain_handlers = {};
 	-- Loop through the chains in the parsed ruleset (e.g. incoming, outgoing)
 	end
 	return chain_handlers;
 end
+local function compile_firewall_rules(filename)
+	local ruleset, err = parse_firewall_rules(filename);
+	if not ruleset then return nil, err; end
+	local chain_handlers = process_firewall_rules(ruleset);
+	return chain_handlers;
+end
 local function compile_handler(code_string, filename)
 	-- Prepare event handler function
 	local chunk, err = loadstring(code_string, "="..filename);
 	if not chunk then
 		return nil, "Error compiling (probably a compiler bug, please report): "..err;