Fix handling of 0 for Backup Count. Fixes #7273

1 files changed, 18 insertions, 8 deletions

diff --git a/src/usr/local/www/diag_confbak.php b/src/usr/local/www/diag_confbak.php
index 5c3e3c9..faed2ef 100644
--- a/src/usr/local/www/diag_confbak.php
+++ b/src/usr/local/www/diag_confbak.php
@@ -30,15 +30,20 @@
 require_once("guiconfig.inc");
 
 if (isset($_POST['backupcount'])) {
-	if (is_numericint($_POST['backupcount'])) {
-		$config['system']['backupcount'] = $_POST['backupcount'];
-		$changedescr = $config['system']['backupcount'];
-	} else {
-		unset($config['system']['backupcount']);
-		$changedescr = gettext("(platform default)");
+	if (!empty($_POST['backupcount']) && (!is_numericint($_POST['backupcount']) || ($_POST['backupcount'] < 0))) {
+		$input_errors[] = gettext("Invalid Backup Count specified");
 	}
 
-	write_config(sprintf(gettext("Changed backup revision count to %s"), $changedescr));
+	if (!$input_errors) {
+		if (is_numericint($_POST['backupcount'])) {
+			$config['system']['backupcount'] = $_POST['backupcount'];
+			$changedescr = $config['system']['backupcount'];
+		} elseif (empty($_POST['backupcount'])) {
+			unset($config['system']['backupcount']);
+			$changedescr = gettext("(platform default)");
+		}
+		write_config(sprintf(gettext("Changed backup revision count to %s"), $changedescr));
+	}
 }
 
 $confvers = unserialize(file_get_contents($g['cf_conf_path'] . '/backup/backup.cache'));
@@ -97,6 +102,10 @@ $pgtitle = array(gettext("Diagnostics"), htmlspecialchars(gettext("Backup & Rest
 $pglinks = array("", "diag_backup.php", "@self");
 include("head.inc");
 
+if ($input_errors) {
+	print_input_errors($input_errors);
+}
+
 if ($savemsg) {
 	print_info_box($savemsg, 'success');
 }
@@ -154,7 +163,8 @@ $section->addInput(new Form_Input(
 	'backupcount',
 	'Backup Count',
 	'number',
-	$config['system']['backupcount']
+	$config['system']['backupcount'],
+	['min' => '0']
 ))->setHelp('Maximum number of old configurations to keep in the cache, 0 for no backups, or leave blank for the default value (%s for the current platform).', $g['default_config_backup_count']);
 
 $space = exec("/usr/bin/du -sh /conf/backup | /usr/bin/awk '{print $1;}'");