Woops. Two functions are using $retval. Insert our rule error detection code between the two.

1 files changed, 4 insertions, 4 deletions

diff --git a/usr/local/www/system_advanced.php b/usr/local/www/system_advanced.php
index 176003e..cc4610e 100755
--- a/usr/local/www/system_advanced.php
+++ b/usr/local/www/system_advanced.php
@@ -129,13 +129,13 @@ if ($_POST) {
 		if (!file_exists($d_sysrebootreqd_path)) {
 			config_lock();
 			$retval = filter_configure();
+			if(stristr($retval, "error") <> true)
+			    $savemsg = get_std_save_message($retval);
+			else
+			    $savemsg = $retval;
 			$retval |= interfaces_optional_configure();
 			config_unlock();
 		}
-		if(stristr($retval, "error") <> true)
-		    $savemsg = get_std_save_message($retval);
-		else
-		    $savemsg = $retval;
 
 		/* Setup pf rules since the user may have changed the optimization value */
 		config_lock();