modules/admin-full: whitesapce cleanup in routes.lua

libs/sys:		code changes in routes6() and defaultroute6()

1 files changed, 40 insertions, 41 deletions

diff --git a/libs/sys/luasrc/sys.lua b/libs/sys/luasrc/sys.lua
index e92adc1ad..fee1feeaf 100644
--- a/libs/sys/luasrc/sys.lua
+++ b/libs/sys/luasrc/sys.lua
@@ -319,14 +319,15 @@ end
 --			{ "source", "dest", "nexthop", "metric", "refcount", "usecount",
 --			  "flags", "device" }
 function net.defaultroute6()
-	local route = nil
+	local route   = nil
 	local routes6 = net.routes6()
-	if not routes6 then
-		return nil
-	end
-	for _, r in pairs(routes6) do
-		if r.dest:prefix() == 0 and (not route or route.metric > r.metric) then
-			route = r
+	if routes6 then
+		for _, r in pairs(routes6) do
+			if r.dest:prefix() == 0 and
+			   (not route or route.metric > r.metric)
+			then
+				route = r
+			end
 		end
 	end
 	return route
@@ -419,46 +420,44 @@ end
 --			{ "source", "dest", "nexthop", "metric", "refcount", "usecount",
 --			  "flags", "device" }
 function net.routes6()
-	local routes = { }
-
-	if not luci.fs.access("/proc/net/ipv6_route", "r") then
-		return nil
-	end
+	if luci.fs.access("/proc/net/ipv6_route", "r") then
+		local routes = { }
+
+		for line in io.lines("/proc/net/ipv6_route") do
+
+			local dst_ip, dst_prefix, src_ip, src_prefix, nexthop,
+				  metric, refcnt, usecnt, flags, dev = line:match(
+				"([a-f0-9]+) ([a-f0-9]+) " ..
+				"([a-f0-9]+) ([a-f0-9]+) " ..
+				"([a-f0-9]+) ([a-f0-9]+) " ..
+				"([a-f0-9]+) ([a-f0-9]+) " ..
+				"([a-f0-9]+) +([^%s]+)"
+			)
 
-	for line in io.lines("/proc/net/ipv6_route") do
+			src_ip = luci.ip.Hex(
+				src_ip, tonumber(src_prefix, 16), luci.ip.FAMILY_INET6, false
+			)
 
-		local dst_ip, dst_prefix, src_ip, src_prefix, nexthop,
-			  metric, refcnt, usecnt, flags, dev = line:match(
-			"([a-f0-9]+) ([a-f0-9]+) " ..
-			"([a-f0-9]+) ([a-f0-9]+) " ..
-			"([a-f0-9]+) ([a-f0-9]+) " ..
-			"([a-f0-9]+) ([a-f0-9]+) " ..
-			"([a-f0-9]+) +([^%s]+)"
-		)
+			dst_ip = luci.ip.Hex(
+				dst_ip, tonumber(dst_prefix, 16), luci.ip.FAMILY_INET6, false
+			)
 
-		src_ip = luci.ip.Hex(
-			src_ip, tonumber(src_prefix, 16), luci.ip.FAMILY_INET6, false
-		)
+			nexthop = luci.ip.Hex( nexthop, 128, luci.ip.FAMILY_INET6, false )
 
-		dst_ip = luci.ip.Hex(
-			dst_ip, tonumber(dst_prefix, 16), luci.ip.FAMILY_INET6, false
-		)
+			routes[#routes+1] = {
+				source   = src_ip,
+				dest     = dst_ip,
+				nexthop  = nexthop,
+				metric   = tonumber(metric, 16),
+				refcount = tonumber(refcnt, 16),
+				usecount = tonumber(usecnt, 16),
+				flags    = tonumber(flags, 16),
+				device   = dev
+			}
+		end
 
-		nexthop = luci.ip.Hex( nexthop, 128, luci.ip.FAMILY_INET6, false )
-
-		routes[#routes+1] = {
-			source   = src_ip,
-			dest     = dst_ip,
-			nexthop  = nexthop,
-			metric   = tonumber(metric, 16),
-			refcount = tonumber(refcnt, 16),
-			usecount = tonumber(usecnt, 16),
-			flags    = tonumber(flags, 16),
-			device   = dev
-		}
+		return routes
 	end
-
-	return routes
 end
 
 --- Tests whether the given host responds to ping probes.