Fix bad argument #1 to 'tonumber'
authorBrian J. Murrell <brian@interlinx.bc.ca>
Mon, 4 Dec 2017 13:44:00 +0000 (08:44 -0500)
committerYousong Zhou <yszhou4tech@gmail.com>
Fri, 26 Jan 2018 10:31:19 +0000 (18:31 +0800)
commitf8682164c1510b21a500461ca365a61d8eccb724
tree4322cb7465ec3ce0efbf85f026dc02363ad13aa1
parent1472b81a005583944e07047a6da5ceb5d8dba219
Fix bad argument #1 to 'tonumber'

Fixes [FS#1118](https://bugs.lede-project.org/index.php?do=details&task_id=1118)

Luci chokes on devices that do not have _iptables_ installed with:

/usr/lib/lua/luci/dispatcher.lua:460: Failed to execute function dispatcher target for entry '/'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:460: Failed to execute firstchild dispatcher target for entry '/admin'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:460: Failed to execute function dispatcher target for entry '/admin/status'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:460: Failed to execute template dispatcher target for entry '/admin/status/overview'.
The called action terminated with an exception:
/usr/lib/lua/luci/template.lua:97: Failed to execute template 'admin_status/index'.
A runtime error occured: [string "/usr/lib/lua/luci/view/admin_status/index.h..."]:34: bad argument #1 to 'tonumber' (string expected, got nil)
stack traceback:
[C]: in function 'assert'
/usr/lib/lua/luci/dispatcher.lua:460: in function 'dispatch'
/usr/lib/lua/luci/dispatcher.lua:141: in function </usr/lib/lua/luci/dispatcher.lua:140>

The code should be more defensive.
modules/luci-mod-admin-full/luasrc/view/admin_status/index.htm