applications/luci-splash: fix another crash in luci-splash when there were empty...
authorManuel Munz <freifunk@somakoma.de>
Mon, 11 Nov 2013 20:12:40 +0000 (20:12 +0000)
committerManuel Munz <freifunk@somakoma.de>
Mon, 11 Nov 2013 20:12:40 +0000 (20:12 +0000)
applications/luci-splash/root/usr/sbin/luci-splash

index 25a12f6a43450a15e6d9f845ee8f82172f1dc216..6b1e41edbde7c63ec07c38da766476fce14a3768 100755 (executable)
@@ -291,21 +291,28 @@ function get_known_macs(list)
        local leased_macs = { }
 
        if not list or list == "lease" then
-
-               uci:foreach("luci_splash_leases", "lease",
-                       function(s) leased_macs[s.mac:lower()] = true end)
+               uci:foreach("luci_splash_leases", "lease", function(s)
+                       if s.mac then
+                               leased_macs[s.mac:lower()] = true
+                       end
+               end)
        end
 
        if not list or list == "whitelist" then
-               uci:foreach("luci_splash", "whitelist",
-                       function(s) leased_macs[s.mac:lower()] = true end)
+               uci:foreach("luci_splash", "whitelist", function(s)
+                       if s.mac then
+                               leased_macs[s.mac:lower()] = true
+                       end
+               end)
        end
 
        if not list or list == "blacklist" then
-               uci:foreach("luci_splash", "blacklist",
-                       function(s) leased_macs[s.mac:lower()] = true end)
+               uci:foreach("luci_splash", "blacklist", function(s)
+                       if s.mac then
+                               leased_macs[s.mac:lower()] = true
+                       end
+               end)
        end
-
        return leased_macs
 end