* luci/app/firewall:

	- remove empty value from src zone, param is mandatory
	- add empty value to outgoing zone to allow removal of this param
	- change default action to "accept" to prevent unwanted lockouts

diff --git a/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua b/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua
index 8ff03b02df6193f7136f59960c98033cb010360e..0d9af93fc6efd79f8071181b51ffb66eb2ffb3ec 100644 (file)
--- a/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua
+++ b/applications/luci-fw/luasrc/model/cbi/luci_fw/firewall.lua
@@ -18,10 +18,10 @@ s.addremove = true
 s.anonymous = true
 
 iface = s:option(ListValue, "src")
-iface:value("")
 iface.rmempty = true
 
 oface = s:option(ListValue, "dest")
+oface:value("")
 oface.optional = true
 
 luci.model.uci.cursor():foreach("firewall", "zone",
@@ -56,6 +56,7 @@ dport:depends("proto", "tcpudp")
 
 jump = s:option(ListValue, "target")
 jump.rmempty = true
+jump.default = "ACCEPT"
 jump:value("DROP", translate("fw_drop"))
 jump:value("ACCEPT", translate("fw_accept"))
 jump:value("REJECT", translate("fw_reject"))