From 08cc7f673c377bf88897743e340097e93f1e95f4 Mon Sep 17 00:00:00 2001 From: ftimme <ft@falkotimme.com> Date: Wed, 16 Jan 2013 09:30:05 -0500 Subject: [PATCH] - Changed regex for redirect path (Web sites, subdomains, vhost subdomains, alias domains) and web folder (vhost subdomains) so that ".." is not allowed (in order to prevent path traversals). - nginx: don't allow folders for proxy redirects (subdomains and alias domains); URL is required. - nginx: modified rewriting. --- server/plugins-available/cron_plugin.inc.php | 23 ++++++++++++----------- 1 files changed, 12 insertions(+), 11 deletions(-) diff --git a/server/plugins-available/cron_plugin.inc.php b/server/plugins-available/cron_plugin.inc.php index d1e31d1..4ccbcb2 100644 --- a/server/plugins-available/cron_plugin.inc.php +++ b/server/plugins-available/cron_plugin.inc.php @@ -142,7 +142,7 @@ } // make temp directory writable for the apache and website users - chmod(escapeshellcmd($parent_domain["document_root"].'/tmp'), 0777); + $app->system->chmod(escapeshellcmd($parent_domain["document_root"].'/tmp'), 0777); /** TODO READ CRON MASTER **/ @@ -183,8 +183,9 @@ //* try to find customer's mail address /** TODO: add possibility for client to choose mail notification! **/ - $cron_content = "MAILTO=''\n\n"; - $chr_cron_content = "MAILTO=''\n\n"; + $cron_content = "MAILTO=''\n"; + $cron_content .= "SHELL='/bin/sh'\n\n"; + $chr_cron_content = "MAILTO=''\n"; $chr_cron_content .= "SHELL='/usr/sbin/jk_chrootsh'\n\n"; $cmd_count = 0; @@ -197,11 +198,11 @@ if($job['run_month'] == '@reboot') { $command = "@reboot"; } else { - $command = "{$job['run_min']}\t{$job['run_hour']}\t{$job['run_mday']}\t{$job['run_month']}\t{$job['run_wday']}"; + $command = str_replace(" ", "", $job['run_min']) . "\t" . str_replace(" ", "", $job['run_hour']) . "\t" . str_replace(" ", "", $job['run_mday']) . "\t" . str_replace(" ", "", $job['run_month']) . "\t" . str_replace(" ", "", $job['run_wday']); } $command .= "\t{$this->parent_domain['system_user']}"; //* running as user if($job['type'] == 'url') { - $command .= "\t{$cron_config['wget']} -q -O /dev/null " . escapeshellarg($job['command']) . " >/dev/null 2>&1"; + $command .= "\t{$cron_config['wget']} -q -t 1 -T 7200 -O /dev/null " . escapeshellarg($job['command']) . " >/dev/null 2>&1"; } else { if($job['type'] == 'chrooted') { if(substr($job['command'], 0, strlen($this->parent_domain['document_root'])) == $this->parent_domain['document_root']) { @@ -216,10 +217,10 @@ } if($job['type'] == 'chrooted') { - $chr_cron_content .= $command . "\t#{$job['domain']}\n"; + $chr_cron_content .= $command . " #{$job['domain']}\n"; $chr_cmd_count++; } else { - $cron_content .= $command . "\t#{$job['domain']}\n"; + $cron_content .= $command . " #{$job['domain']}\n"; $cmd_count++; } } @@ -233,19 +234,19 @@ } if($cmd_count > 0) { - file_put_contents($cron_file, $cron_content); + $app->system->file_put_contents($cron_file, $cron_content); $app->log("Wrote Cron file $cron_file with content:\n$cron_content",LOGLEVEL_DEBUG); } else { - @unlink($cron_file); + $app->system->unlink($cron_file); $app->log("Deleted Cron file $cron_file",LOGLEVEL_DEBUG); } $cron_file = escapeshellcmd($cron_config["crontab_dir"].'/ispc_chrooted_'.$this->parent_domain["system_user"]); if($chr_cmd_count > 0) { - file_put_contents($cron_file, $chr_cron_content); + $app->system->file_put_contents($cron_file, $chr_cron_content); $app->log("Wrote Cron file $cron_file with content:\n$chr_cron_content",LOGLEVEL_DEBUG); } else { - @unlink($cron_file); + $app->system->unlink($cron_file); $app->log("Deleted Cron file $cron_file",LOGLEVEL_DEBUG); } -- Gitblit v1.9.1