firewall: two more group issues

opnsense · Aug 1, 2023 · 0e1aa4b · 0e1aa4b
1 parent e27d37a
commit 0e1aa4b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/www/firewall_rules.php b/src/www/firewall_rules.php
@@ -736,7 +736,7 @@ function filter_rule_item_alias_tooltip($alias_name)
 <?php
                 $ifgroups = [];
                 foreach (config_read_array('ifgroups', 'ifgroupentry') as $ifgroup) {
-                    if (!empty($ifgroup['members']) && in_array($selected_if, explode(' ', $ifgroup['members']))) {
+                    if (!empty($ifgroup['members']) && in_array($selected_if, preg_split('/[ |,]+/', $ifgroup['members']))) {
                         $ifgroups[] = $ifgroup['ifname'];
                     }
                 }

diff --git a/src/www/interfaces_assign.php b/src/www/interfaces_assign.php
@@ -42,7 +42,7 @@ function link_interface_to_group($int)
 
     if (isset($config['ifgroups']['ifgroupentry'])) {
         foreach ($config['ifgroups']['ifgroupentry'] as $group) {
-            if (in_array($int, explode(" ", $group['members']))) {
+            if (in_array($int, preg_split('/[ |,]+/', $group['members']))) {
                 $result[$group['ifname']] = $int;
             }
         }