Do not attempt to remove interfaces from CP zone, captiveportal_configure_zone() will take care of it

1 files changed, 1 insertions, 13 deletions

diff --git a/src/usr/local/www/services_captiveportal.php b/src/usr/local/www/services_captiveportal.php
index 5887bd5..cdfbebd 100644
--- a/src/usr/local/www/services_captiveportal.php
+++ b/src/usr/local/www/services_captiveportal.php
@@ -341,7 +341,6 @@ if ($_POST) {
 
 			$cpzoneid = $newcp['zoneid'];
 		}
-		$oldifaces = explode(",", $newcp['interface']);
 		if (is_array($_POST['cinterface'])) {
 			$newcp['interface'] = implode(",", $_POST['cinterface']);
 		}
@@ -449,19 +448,8 @@ if ($_POST) {
 
 		write_config();
 
-		/* Clear up unselected interfaces */
-		$newifaces = explode(",", $newcp['interface']);
-		$toremove = array_diff($oldifaces, $newifaces);
-
-		if (!empty($toremove)) {
-			foreach ($toremove as $removeif) {
-				$removeif = get_real_interface($removeif);
-				mwexec("/sbin/ipfw zone {$cpzoneid} mdel {$removeif}");
-			}
-		}
-
 		captiveportal_configure_zone($newcp);
-		unset($newcp, $newifaces, $toremove);
+		unset($newcp);
 		filter_configure();
 		header("Location: services_captiveportal_zones.php");
 		exit;