diff options
author | Ermal <eri@pfsense.org> | 2012-07-14 16:33:54 +0000 |
---|---|---|
committer | Ermal <eri@pfsense.org> | 2012-07-14 16:33:54 +0000 |
commit | 1dc9e022904cf22326f66de07f1a6f43dac8aacd (patch) | |
tree | 0bf80d41725591463eeb163911bf8118ec7030b2 /config/snort/snort.inc | |
parent | f03281254a3acefa5a05eab388af6295a4222ae1 (diff) | |
download | pfsense-packages-1dc9e022904cf22326f66de07f1a6f43dac8aacd.tar.gz pfsense-packages-1dc9e022904cf22326f66de07f1a6f43dac8aacd.tar.bz2 pfsense-packages-1dc9e022904cf22326f66de07f1a6f43dac8aacd.zip |
Actually restrict to the gateway where the instance is listening
Diffstat (limited to 'config/snort/snort.inc')
-rw-r--r-- | config/snort/snort.inc | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/config/snort/snort.inc b/config/snort/snort.inc index 20cae53c..f6181819 100644 --- a/config/snort/snort.inc +++ b/config/snort/snort.inc @@ -97,7 +97,7 @@ function snort_find_list($find_name, $type = 'whitelist') { } /* func builds custom whitelests */ -function snort_build_list($listname = "") { +function snort_build_list($snortcfg, $listname = "") { global $config, $g; /* Add loopback to whitelist (ftphelper) */ @@ -140,12 +140,6 @@ function snort_build_list($listname = "") { if (is_ipaddr($subnet)) { $home_net .= "{$subnet} "; } - if ($wangw == 'yes') { - /* Add Gateway on WAN interface to whitelist (For RRD graphs) */ - $gw = get_interface_gateway($int); - if (is_ipaddr($gw)) - $home_net .= "{$gw} "; - } } else { $ifcfg = $config['interfaces'][$int]; switch ($ifcfg['ipaddr']) { @@ -174,6 +168,12 @@ function snort_build_list($listname = "") { } } + if ($wangw == 'yes') { + $gw = get_interface_gateway($snortcfg['interface']); + if (is_ipaddr($gw)) + $home_net .= "{$gw} "; + } + if ($wandns == 'yes') { /* Add DNS server for WAN interface to whitelist */ $dns_servers = get_dns_servers(); @@ -979,12 +979,12 @@ function snort_generate_conf($snortcfg) { $snortcfgdir = "{$snortdir}/snort_{$snort_uuid}_{$if_real}"; /* custom home nets */ - $home_net_list = snort_build_list($snortcfg['homelistname']); + $home_net_list = snort_build_list($snortcfg, $snortcfg['homelistname']); $home_net = implode(",", $home_net_list); $external_net = '!$HOME_NET'; if (!empty($snortcfg['externallistname']) && $snortcfg['externallistname'] != 'default') { - $external_net_list = snort_build_list($snortcfg['externallistname']); + $external_net_list = snort_build_list($snortcfg, $snortcfg['externallistname']); $external_net = implode(",", $external_net_list); } @@ -1028,7 +1028,7 @@ function snort_generate_conf($snortcfg) { $pfkill = ""; if ($snortcfg['blockoffenderskill'] == "on") $pfkill = "kill"; - $spoink_wlist = snort_build_list($snortcfg['whitelistname']); + $spoink_wlist = snort_build_list($snortcfg, $snortcfg['whitelistname']); /* write whitelist */ @file_put_contents("{$snortcfgdir}/{$snortcfg['whitelistname']}", implode("\n", $spoink_wlist)); $spoink_type = "output alert_pf: {$snortcfgdir}/{$snortcfg['whitelistname']},snort2c,{$snortcfg['blockoffendersip']},{$pfkill}"; |