aboutsummaryrefslogtreecommitdiffstats
path: root/config
diff options
context:
space:
mode:
authorrjcrowder <randyj.crowder@gmail.com>2012-10-31 20:35:00 -0300
committerrjcrowder <randyj.crowder@gmail.com>2012-10-31 20:35:00 -0300
commit7953f4305d0d5e843dc75371f48fc62c6a636d51 (patch)
tree24c64d4db907b223158a65b06fe52d3f2b27ebaa /config
parent21b5080d9b3bbf19d6441aebea87fe35f29783ca (diff)
downloadpfsense-packages-7953f4305d0d5e843dc75371f48fc62c6a636d51.tar.gz
pfsense-packages-7953f4305d0d5e843dc75371f48fc62c6a636d51.tar.bz2
pfsense-packages-7953f4305d0d5e843dc75371f48fc62c6a636d51.zip
Update config/dansguardian/dansguardian.php
Added return code check when fetching the blacklists. In the previous version, the blacklist directories were removed if the fetch failed. Once the directories were removed, dansguardian would not start because the config files referenced blacklist files that did not exist. This version does not remove the blacklist directories if the blacklist fetch fails.
Diffstat (limited to 'config')
-rw-r--r--config/dansguardian/dansguardian.php38
1 files changed, 22 insertions, 16 deletions
diff --git a/config/dansguardian/dansguardian.php b/config/dansguardian/dansguardian.php
index 8571e1b7..675db6c7 100644
--- a/config/dansguardian/dansguardian.php
+++ b/config/dansguardian/dansguardian.php
@@ -39,28 +39,34 @@ require_once("/etc/inc/pkg-utils.inc");
require_once("/etc/inc/globals.inc");
require_once("/usr/local/pkg/dansguardian.inc");
-function fetch_blacklist(){
+function fetch_blacklist() {
global $config,$g;
$url=$config['installedpackages']['dansguardianblacklist']['config'][0]['url'];
- if (is_url($url)){
+ if (is_url($url)) {
conf_mount_rw();
print "file download start..";
unlink_if_exists("/usr/local/etc/dansguardian/lists/blacklist.tgz");
- exec("/usr/bin/fetch -o /usr/local/etc/dansguardian/lists/blacklist.tgz ".escapeshellarg($url));
- chdir ("/usr/local/etc/dansguardian/lists");
- if (is_dir ("blacklists.old"))
- exec ('rm -rf /usr/local/etc/dansguardian/lists/blacklists.old');
- rename("blacklists","blacklists.old");
- exec('/usr/bin/tar -xvzf /usr/local/etc/dansguardian/lists/blacklist.tgz 2>&1',$output,$return);
- if (preg_match("/x\W+(\w+)/",$output[0],$matches)){
- if ($matches[1] != "blacklists")
- rename("./".$matches[1],"blacklists");
- read_lists();
+ exec("/usr/bin/fetch -o /usr/local/etc/dansguardian/lists/blacklist.tgz ".escapeshellarg($url),$output,$return);
+ if ($return == 0) {
+ chdir ("/usr/local/etc/dansguardian/lists");
+ if (is_dir ("blacklists.old"))
+ exec ('rm -rf /usr/local/etc/dansguardian/lists/blacklists.old');
+ rename("blacklists","blacklists.old");
+ exec('/usr/bin/tar -xvzf /usr/local/etc/dansguardian/lists/blacklist.tgz 2>&1',$output,$return);
+ if (preg_match("/x\W+(\w+)/",$output[0],$matches)) {
+ if ($matches[1] != "blacklists")
+ rename("./".$matches[1],"blacklists");
+ read_lists();
}
- else
- file_notice("Dansguardian - Could not determine Blacklist extract dir. Categories not updated","");
- }
- else{
+ else {
+ file_notice("Dansguardian - Could not determine Blacklist extract dir. Categories not updated","");
+ }
+ }
+ else {
+ file_notice("Dansguardian - Could not fetch blacklists from url","");
+ }
+ }
+ else {
file_notice("Dansguardian - Blacklist url is invalid.","");
}
}