modules/admin-*: Fixes for firmware upgrade and system reset

4 files changed, 16 insertions, 6 deletions

diff --git a/modules/admin-full/luasrc/controller/admin/system.lua b/modules/admin-full/luasrc/controller/admin/system.lua
index 01663a4c3..8e3a3d799 100644
--- a/modules/admin-full/luasrc/controller/admin/system.lua
+++ b/modules/admin-full/luasrc/controller/admin/system.lua
@@ -155,7 +155,7 @@ function action_packages()
 end
 
 function action_backup()
-	local reset_avail = luci.sys.exec([[grep '"rootfs_data"' /proc/mtd >/dev/null 2>&1]]) == 0
+	local reset_avail = os.execute([[grep '"rootfs_data"' /proc/mtd >/dev/null 2>&1]]) == 0
 	local restore_cmd = "gunzip | tar -xC/ >/dev/null 2>&1"
 	local backup_cmd  = "tar -c %s | gzip 2>/dev/null"
 	
@@ -242,7 +242,10 @@ function action_upgrade()
 
 	local ret  = nil
 	local plat = luci.fs.mtime("/lib/upgrade/platform.sh")
+	local broadcom = os.execute('grep brcm_ /lib/upgrade/platform.sh >/dev/null 2>&1') == 0
 	local tmpfile = "/tmp/firmware.img"
+	
+	local keep_avail = not broadcom
 
 	local file
 	luci.http.setfilehandler(
@@ -260,13 +263,13 @@ function action_upgrade()
 	)
 
 	local fname   = luci.http.formvalue("image")
-	local keepcfg = luci.http.formvalue("keepcfg")
+	local keepcfg = keep_avail and luci.http.formvalue("keepcfg")
 
 	if plat and fname then
 		ret = luci.sys.flash(tmpfile, keepcfg and _keep_pattern())
 	end
 
-	luci.template.render("admin_system/upgrade", {sysupgrade=plat, ret=ret})
+	luci.template.render("admin_system/upgrade", {sysupgrade=plat, ret=ret, keep_avail=keep_avail})
 end
 
 function _keep_pattern()
diff --git a/modules/admin-full/luasrc/view/admin_system/upgrade.htm b/modules/admin-full/luasrc/view/admin_system/upgrade.htm
index 350d6a8db..e0fbb9332 100644
--- a/modules/admin-full/luasrc/view/admin_system/upgrade.htm
+++ b/modules/admin-full/luasrc/view/admin_system/upgrade.htm
@@ -24,8 +24,10 @@ $Id$
 		<input type="file" size="30" name="image" />
 		<br />
 		<br />
+		<% if keep_avail then -%>
 		<input type="checkbox" name="keepcfg" value="1" checked="checked" />
 		<span class="bold"><%:a_s_flash_keepcfg%></span>
+		<% end -%>
 	</div>
 	<div>
 		<input type="submit" value="<%:a_s_flash_fwupgrade%>" />
diff --git a/modules/admin-mini/luasrc/controller/mini/system.lua b/modules/admin-mini/luasrc/controller/mini/system.lua
index e272a1fc7..ef1489b9b 100644
--- a/modules/admin-mini/luasrc/controller/mini/system.lua
+++ b/modules/admin-mini/luasrc/controller/mini/system.lua
@@ -28,7 +28,7 @@ function index()
 end
 
 function action_backup()
-	local reset_avail = luci.sys.exec([[grep '"rootfs_data"' /proc/mtd >/dev/null 2>&1]]) == 0
+	local reset_avail = os.execute([[grep '"rootfs_data"' /proc/mtd >/dev/null 2>&1]]) == 0
 	local restore_cmd = "gunzip | tar -xC/ >/dev/null 2>&1"
 	local backup_cmd  = "tar -c %s | gzip 2>/dev/null"
 	
@@ -81,7 +81,10 @@ function action_upgrade()
 
 	local ret  = nil
 	local plat = luci.fs.mtime("/lib/upgrade/platform.sh")
+	local broadcom = os.execute('grep brcm_ /lib/upgrade/platform.sh >/dev/null 2>&1') == 0
 	local tmpfile = "/tmp/firmware.img"
+	
+	local keep_avail = not broadcom
 
 	local file
 	luci.http.setfilehandler(
@@ -99,13 +102,13 @@ function action_upgrade()
 	)
 
 	local fname   = luci.http.formvalue("image")
-	local keepcfg = luci.http.formvalue("keepcfg")
+	local keepcfg = keep_avail and luci.http.formvalue("keepcfg")
 
 	if plat and fname then
 		ret = luci.sys.flash(tmpfile, keepcfg and _keep_pattern())
 	end
 
-	luci.template.render("mini/upgrade", {sysupgrade=plat, ret=ret})
+	luci.template.render("mini/upgrade", {sysupgrade=plat, ret=ret, keep_avail=keep_avail})
 end
 
 function action_passwd()
diff --git a/modules/admin-mini/luasrc/view/mini/upgrade.htm b/modules/admin-mini/luasrc/view/mini/upgrade.htm
index 350d6a8db..e0fbb9332 100644
--- a/modules/admin-mini/luasrc/view/mini/upgrade.htm
+++ b/modules/admin-mini/luasrc/view/mini/upgrade.htm
@@ -24,8 +24,10 @@ $Id$
 		<input type="file" size="30" name="image" />
 		<br />
 		<br />
+		<% if keep_avail then -%>
 		<input type="checkbox" name="keepcfg" value="1" checked="checked" />
 		<span class="bold"><%:a_s_flash_keepcfg%></span>
+		<% end -%>
 	</div>
 	<div>
 		<input type="submit" value="<%:a_s_flash_fwupgrade%>" />