From 85b7eb644dec64190b67207e09b52dcedc1344b6 Mon Sep 17 00:00:00 2001 From: Ermal Date: Wed, 3 Aug 2011 17:28:28 +0000 Subject: Remove uneccessary calls to write_config() and add some more checks. --- config/snort/snort.inc | 36 +++++++------------------------- config/snort/snort_interfaces_global.php | 14 ++++++++++--- 2 files changed, 18 insertions(+), 32 deletions(-) (limited to 'config/snort') diff --git a/config/snort/snort.inc b/config/snort/snort.inc index cbbebf26..7a5a4ffb 100644 --- a/config/snort/snort.inc +++ b/config/snort/snort.inc @@ -484,12 +484,12 @@ function snort_postinstall() { global $config, $g, $snort_pfsense_basever, $snort_arch; - conf_mount_rw(); - /* snort -> advanced features */ - $bpfbufsize = $config['installedpackages']['snortglobal']['bpfbufsize']; - $bpfmaxbufsize = $config['installedpackages']['snortglobal']['bpfmaxbufsize']; - $bpfmaxinsns = $config['installedpackages']['snortglobal']['bpfmaxinsns']; + if (is_array($config['installedpackages']['snortglobal'])) { + $bpfbufsize = $config['installedpackages']['snortglobal']['bpfbufsize']; + $bpfmaxbufsize = $config['installedpackages']['snortglobal']['bpfmaxbufsize']; + $bpfmaxinsns = $config['installedpackages']['snortglobal']['bpfmaxinsns']; + } /* cleanup default files */ @unlink('/usr/local/etc/snort/snort.conf-sample'); @@ -631,7 +631,7 @@ function snort_postinstall() } /* make sure snort-old is deinstalled */ - unset($config['installedpackages']['snort'], $config['installedpackages']['snortdefservers'], $config['installedpackages']['snortwhitelist']); + unset($config['installedpackages']['snortdefservers'], $config['installedpackages']['snortwhitelist']); unset($config['installedpackages']['snortthreshold'], $config['installedpackages']['snortadvanced']); /* remake saved settings */ @@ -641,8 +641,6 @@ function snort_postinstall() sync_snort_package_empty(); update_output_window(gettext("Finnished Rebuilding files...")); } - - conf_mount_ro(); } function sync_package_snort_reinstall() @@ -980,22 +978,7 @@ function sync_snort_package() $snortloglimitsize = $config['installedpackages']['snortglobal']['snortloglimitsize']; $snortloglimit = $config['installedpackages']['snortglobal']['snortloglimit']; - if ($snortloglimit == '') - /* code will set limit to 21% of slice that is unused */ - $config['installedpackages']['snortglobal']['snortloglimit'] = 'on'; - - if ($snortloglimitsize == '') { - /* code will set limit to 21% of slice that is unused */ - $snortloglimitDSKsize = round(exec('df -k /var | grep -v "Filesystem" | awk \'{print $4}\'') * .22 / 1024); - $config['installedpackages']['snortglobal']['snortloglimitsize'] = $snortloglimitDSKsize; - } - snort_snortloglimit_install_cron($config['installedpackages']['snortglobal']['snortloglimit'] == 'on' ? true : false); - - /* XXX: Really need write_config here? */ - write_config(); - /* XXX: Restore rw mode since write_config sets ro */ - conf_mount_rw(); } /* only run when a single iface needs to sync */ @@ -1589,8 +1572,7 @@ function create_snort_conf($id, $if_real, $snort_uuid) } } -function snort_deinstall() -{ +function snort_deinstall() { global $config, $g; /* remove custom sysctl */ @@ -1642,9 +1624,6 @@ function snort_deinstall() /* Keep this as a last step */ if ($config['installedpackages']['snortglobal']['forcekeepsettings'] != 'on') unset($config['installedpackages']['snortglobal']); - - write_config(); /* XXX */ - conf_mount_rw(); } function generate_snort_conf($id, $if_real, $snort_uuid) @@ -1681,7 +1660,6 @@ function generate_snort_conf($id, $if_real, $snort_uuid) exec("/bin/cp /usr/local/etc/snort/sid-msg.map /usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/sid-msg.map"); exec("/bin/cp /usr/local/etc/snort/unicode.map /usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/unicode.map"); exec("/bin/cp /usr/local/etc/snort/threshold.conf /usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/threshold.conf"); - exec("/bin/cp /usr/local/etc/snort/snort.conf /usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/snort.conf"); exec("/usr/bin/touch /usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/barnyard2.conf"); if (!is_dir("/usr/local/etc/snort/snort_{$snort_uuid}_{$if_real}/rules")) diff --git a/config/snort/snort_interfaces_global.php b/config/snort/snort_interfaces_global.php index 2c0d4404..d9336fd3 100644 --- a/config/snort/snort_interfaces_global.php +++ b/config/snort/snort_interfaces_global.php @@ -62,8 +62,16 @@ if (!$input_errors) { $config['installedpackages']['snortglobal']['oinkmastercode'] = $_POST['oinkmastercode']; $config['installedpackages']['snortglobal']['emergingthreats'] = $_POST['emergingthreats'] ? 'on' : 'off'; $config['installedpackages']['snortglobal']['rm_blocked'] = $_POST['rm_blocked']; - $config['installedpackages']['snortglobal']['snortloglimit'] = $_POST['snortloglimit']; - $config['installedpackages']['snortglobal']['snortloglimitsize'] = $_POST['snortloglimitsize']; + if ($_POST['snortloglimitsize']) { + $config['installedpackages']['snortglobal']['snortloglimit'] = $_POST['snortloglimit']; + $config['installedpackages']['snortglobal']['snortloglimitsize'] = $_POST['snortloglimitsize']; + } else { + $config['installedpackages']['snortglobal']['snortloglimit'] = 'on'; + + /* code will set limit to 21% of slice that is unused */ + $snortloglimitDSKsize = round(exec('df -k /var | grep -v "Filesystem" | awk \'{print $4}\'') * .22 / 1024); + $config['installedpackages']['snortglobal']['snortloglimitsize'] = $snortloglimitDSKsize; + } $config['installedpackages']['snortglobal']['autorulesupdate7'] = $_POST['autorulesupdate7']; $config['installedpackages']['snortglobal']['snortalertlogtype'] = $_POST['snortalertlogtype']; $config['installedpackages']['snortglobal']['forcekeepsettings'] = $_POST['forcekeepsettings'] ? 'on' : 'off'; @@ -323,7 +331,7 @@ enable JavaScript to view this content > + > Enable directory size limit (Default) -- cgit v1.2.3