Commit a203c699 authored by Ad Schellevis's avatar Ad Schellevis

filter, cleanup some more, https://github.com/opnsense/core/pull/1419

parent ac28bc57
...@@ -2044,39 +2044,14 @@ function filter_generate_address(&$FilterIflist, &$rule, $target = 'source', $is ...@@ -2044,39 +2044,14 @@ function filter_generate_address(&$FilterIflist, &$rule, $target = 'source', $is
if (isset($rule[$target]['any'])) { if (isset($rule[$target]['any'])) {
$src = "any"; $src = "any";
} elseif (!empty($rule[$target]['network'])) { } elseif (!empty($rule[$target]['network'])) {
if (strstr($rule[$target]['network'], "opt")) { $network_name = $rule[$target]['network'];
$optmatch = ""; $matches = "";
$matches = ""; if ($network_name == '(self)') {
if (preg_match("/opt([0-9]*)$/", $rule[$target]['network'], $optmatch)) { $src = "(self)";
$src = "({$FilterIflist["opt{$optmatch[1]}"]['if']}:network)"; } elseif (preg_match("/(.*)ip$/", $rule[$target]['network'], $matches)) {
/* check for opt$NUMip here */ $src = "({$FilterIflist["{$matches[1]}"]['if']})";
} elseif (preg_match("/opt([0-9]*)ip/", $rule[$target]['network'], $matches)) {
$src = "({$FilterIflist["opt{$matches[1]}"]['if']})";
}
} else { } else {
switch ($rule[$target]['network']) { $src = "({$FilterIflist[$network_name]['if']}:network)";
case 'wan':
$src = "({$FilterIflist['wan']['if']}:network)";
break;
case 'wanip':
$src = "({$FilterIflist['wan']['if']})";
break;
case 'lan':
$src = "({$FilterIflist['lan']['if']}:network)";
break;
case 'lanip':
$src = "({$FilterIflist['lan']['if']})";
break;
case '(self)':
$src = "(self)";
break;
default:
if (!empty($FilterIflist[$rule[$target]['network']]['if'])) {
$src = "({$FilterIflist[$rule[$target]['network']]['if']}:network)";
} else {
return "";
}
}
} }
if (isset($rule[$target]['not'])) { if (isset($rule[$target]['not'])) {
$src = " !{$src}"; $src = " !{$src}";
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment