luci-base: datatypes.lua: add missing parentheses in conditional
authorJo-Philipp Wich <jo@mein.io>
Tue, 24 Jan 2017 12:51:23 +0000 (13:51 +0100)
committerJo-Philipp Wich <jo@mein.io>
Tue, 24 Jan 2017 12:51:27 +0000 (13:51 +0100)
The missing parens lead to a wrong expression precedence, causing a runtime
error when attempting to compare nil with a number.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
modules/luci-base/luasrc/cbi/datatypes.lua

index 2039a3516b89d53328258ec7e3fe1dc967c12267..98f6a44d77e7929fadce3b0ad1c494bf677f0dcd 100644 (file)
@@ -139,7 +139,7 @@ function ipmask4(val)
        local ip, mask = val:match("^([^/]+)/([^/]+)$")
        local bits = tonumber(mask)
 
-       if bits and bits < 0 or bits > 32 then
+       if bits and (bits < 0 or bits > 32) then
                return false
        end
 
@@ -154,7 +154,7 @@ function ipmask6(val)
        local ip, mask = val:match("^([^/]+)/([^/]+)$")
        local bits = tonumber(mask)
 
-       if bits and bits < 0 or bits > 128 then
+       if bits and (bits < 0 or bits > 128) then
                return false
        end