firewall: validate max length of zone name 451/head
authorHannu Nyman <hannu.nyman@iki.fi>
Wed, 26 Aug 2015 10:51:31 +0000 (13:51 +0300)
committerHannu Nyman <hannu.nyman@iki.fi>
Wed, 26 Aug 2015 10:55:26 +0000 (13:55 +0300)
fw3 sets the maximum length of the zone name to 14 and
ignores zone definitions with too long names.
http://nbd.name/gitweb.cgi?p=firewall3.git;a=blob;f=zones.h;hb=HEAD#l25
http://nbd.name/gitweb.cgi?p=firewall3.git;a=blob;f=zones.c;hb=HEAD#l195

Add a simple validation to ensure that the new zone name is short enough.
This should fix issue #345

Signed-off-by: Hannu Nyman <hannu.nyman@iki.fi>
applications/luci-app-firewall/luasrc/model/cbi/firewall/zone-details.lua

index 3eb95ebb57a916b334c80ced09a568192d1fb465..46b3744403087998a13ee65af7616c8be63b96d4 100644 (file)
@@ -76,6 +76,15 @@ function name.write(self, section, value)
        }
 end
 
+function name.validate(self, value)
+       -- fw3 defines 14 as the maximum length of zone name
+       if #value > 14 then
+               return nil, translate("Zone name is too long")
+       else
+               return value
+       end
+end
+
 p = {
        s:taboption("general", ListValue, "input", translate("Input")),
        s:taboption("general", ListValue, "output", translate("Output")),