If an error occurs during the rule loading, report the error to the user.

4 files changed, 8 insertions, 5 deletions

diff --git a/usr/local/www/firewall_aliases.php b/usr/local/www/firewall_aliases.php
index 770a7df..eaf0639 100755
--- a/usr/local/www/firewall_aliases.php
+++ b/usr/local/www/firewall_aliases.php
@@ -51,7 +51,10 @@ if ($_POST) {
 			$retval = filter_configure();
 			config_unlock();
 		}
-		$savemsg = get_std_save_message($retval);
+		if(stristr($retval, "error") <> true)
+		    $savemsg = get_std_save_message($retval);
+		else
+		    $savemsg = $retval;
 		if ($retval == 0) {
 			if (file_exists($d_aliasesdirty_path))
 				unlink($d_aliasesdirty_path);
diff --git a/usr/local/www/firewall_nat.php b/usr/local/www/firewall_nat.php
index ee1c83a..11e6058 100755
--- a/usr/local/www/firewall_nat.php
+++ b/usr/local/www/firewall_nat.php
@@ -50,7 +50,7 @@ if ($_POST) {
 			$retval |= filter_configure();
 			config_unlock();
 		}
-		if($retval == 0)
+		if(stristr($retval, "error") <> true)
 		    $savemsg = get_std_save_message($retval);
 		else
 		    $savemsg = $retval;
diff --git a/usr/local/www/firewall_rules.php b/usr/local/www/firewall_rules.php
index 0634a66..ca1edc2 100755
--- a/usr/local/www/firewall_rules.php
+++ b/usr/local/www/firewall_rules.php
@@ -60,13 +60,13 @@ if ($_POST) {
 	$pconfig = $_POST;
 
 	if ($_POST['apply']) {
-		$retval = 0;
+		$retval = "Reboot is currently required.";
 		if (!file_exists($d_sysrebootreqd_path)) {
 			config_lock();
 			$retval = filter_configure();
 			config_unlock();
 		}
-		if($retval == 0)
+		if(stristr($retval, "error") <> true)
 		    $savemsg = get_std_save_message($retval);
 		else
 		    $savemsg = $retval;
diff --git a/usr/local/www/firewall_shaper.php b/usr/local/www/firewall_shaper.php
index 267772f..8eb10e8 100755
--- a/usr/local/www/firewall_shaper.php
+++ b/usr/local/www/firewall_shaper.php
@@ -62,7 +62,7 @@ if ($_POST) {
 		config_lock();
 		$retval = filter_configure();
 		config_unlock();
-		if($retval == 0)
+		if(stristr($retval, "error") <> true)
 		    $savemsg = get_std_save_message($retval);
 		else
 		    $savemsg = $retval;