Actually use the previous configured mac/ip, if present, for finding rules to be deleted.

1 files changed, 6 insertions, 3 deletions

diff --git a/usr/local/www/services_captiveportal_ip_edit.php b/usr/local/www/services_captiveportal_ip_edit.php
index cf5145b..57e8476 100755
--- a/usr/local/www/services_captiveportal_ip_edit.php
+++ b/usr/local/www/services_captiveportal_ip_edit.php
@@ -111,10 +111,13 @@ if ($_POST) {
                         $ip['bw_up'] = $_POST['bw_up'];
                 if ($_POST['bw_down'])
                         $ip['bw_down'] = $_POST['bw_down'];
-		if (isset($id) && $a_allowedips[$id])
+		if (isset($id) && $a_allowedips[$id]) {
+			$oldip = $a_allowedips[$id]['ip'];
 			$a_allowedips[$id] = $ip;
-		else
+		} else {
+			$oldip = $ip['ip'];
 			$a_allowedips[] = $ip;
+		}
 		allowedips_sort();
 		
 		write_config();
@@ -122,7 +125,7 @@ if ($_POST) {
 		if (isset($config['captiveportal']['enable']) && is_module_loaded("ipfw.ko")) {
 			$rules = "";
 			for ($i = 3; $i < 10; $i++)
-				$rules .= "table {$i} delete {$ip['ip']}\n";
+				$rules .= "table {$i} delete {$oldip}\n";
 			$rules .= captiveportal_allowedip_configure_entry($ip);
 			file_put_contents("{$g['tmp_path']}/allowedip_tmp{$id}", $rules);
 			mwexec("/sbin/ipfw -q {$g['tmp_path']}/allowedip_tmp{$id}");