Improved input validation

Marius Cramer

2014-07-29 64ea56164ad8f0f2cee5676f84d2d8f064e986e1

 server/lib/classes/system.inc.php

@@ -34,7 +34,9 @@
   var $server_id;
   var $server_conf;
   var $data;

   var $min_uid = 500;
   var $min_gid = 500;
	
   /**
    * Construct for this class
    *
@@ -851,7 +853,7 @@

      //* We allow only some characters in the path
      // * is allowed, for example it is part of wildcard certificates/keys: *.example.com.crt
      if(!preg_match('@^/[-a-zA-Z0-9_/.*~]{1,}$@', $path)) return false;
      if(!preg_match('@^/[-a-zA-Z0-9_/.*]{1,}[~]?$@', $path)) return false;

      //* Check path for symlinks
      $path_parts = explode('/', $path);
@@ -1478,7 +1480,11 @@

   function maildirmake($maildir_path, $user = '', $subfolder = '') {

      global $app;
      global $app, $conf;
		
      // load the server configuration options
      $app->uses("getconf");
      $mail_config = $app->getconf->get_server_config($conf["server_id"], 'mail');

      if($subfolder != '') {
         $dir = escapeshellcmd($maildir_path.'/.'.$subfolder);
@@ -1521,25 +1527,30 @@

      //* Add the subfolder to the subscriptions and courierimapsubscribed files
      if($subfolder != '') {
			
         // Courier
         if(!is_file($maildir_path.'/courierimapsubscribed')) {
            $tmp_file = escapeshellcmd($maildir_path.'/courierimapsubscribed');
            touch($tmp_file);
            chmod($tmp_file, 0744);
            chown($tmp_file, 'vmail');
            chgrp($tmp_file, 'vmail');
         if($mail_config['pop3_imap_daemon'] == 'courier') {
            if(!is_file($maildir_path.'/courierimapsubscribed')) {
               $tmp_file = escapeshellcmd($maildir_path.'/courierimapsubscribed');
               touch($tmp_file);
               chmod($tmp_file, 0744);
               chown($tmp_file, 'vmail');
               chgrp($tmp_file, 'vmail');
            }
            $this->replaceLine($maildir_path.'/courierimapsubscribed', 'INBOX.'.$subfolder, 'INBOX.'.$subfolder, 1, 1);
         }
         $this->replaceLine($maildir_path.'/courierimapsubscribed', 'INBOX.'.$subfolder, 'INBOX.'.$subfolder, 1, 1);

         // Dovecot
         if(!is_file($maildir_path.'/subscriptions')) {
            $tmp_file = escapeshellcmd($maildir_path.'/subscriptions');
            touch($tmp_file);
            chmod($tmp_file, 0744);
            chown($tmp_file, 'vmail');
            chgrp($tmp_file, 'vmail');
         if($mail_config['pop3_imap_daemon'] == 'dovecot') {
            if(!is_file($maildir_path.'/subscriptions')) {
               $tmp_file = escapeshellcmd($maildir_path.'/subscriptions');
               touch($tmp_file);
               chmod($tmp_file, 0744);
               chown($tmp_file, 'vmail');
               chgrp($tmp_file, 'vmail');
            }
            $this->replaceLine($maildir_path.'/subscriptions', $subfolder, $subfolder, 1, 1);
         }
         $this->replaceLine($maildir_path.'/subscriptions', $subfolder, $subfolder, 1, 1);
      }

      $app->log('Created Maildir '.$maildir_path.' with subfolder: '.$subfolder, LOGLEVEL_DEBUG);
@@ -1750,6 +1761,85 @@
      }
   }

   function getapachemodules() {
      global $app;
		
      $cmd = '';
      if(is_installed('apache2ctl')) $cmd = 'apache2ctl -t -D DUMP_MODULES';
      elseif(is_installed('apachectl')) $cmd = 'apachectl -t -D DUMP_MODULES';
      else {
         $app->log("Could not check apache modules, apachectl not found.", LOGLEVEL_WARN);
         return array();
      }
		
      exec($cmd . ' 2>/dev/null', $output, $return_var);
      if($return_var != 0 || !$output[0]) {
         $app->log("Could not check apache modules, apachectl did not return any data.", LOGLEVEL_WARN);
         return array();
      }
		
      $modules = array();
      for($i = 0; $i < count($output); $i++) {
         if(preg_match('/^\s*(\w+)\s+\((shared|static)\)\s*$/', $output[$i], $matches)) {
            $modules[] = $matches[1];
         }
      }
		
      return $modules;
   }
	
   //* ISPConfig mail function
   public function mail($to, $subject, $text, $from, $filepath = '', $filetype = 'application/pdf', $filename = '', $cc = '', $bcc = '', $from_name = '') {
      global $app, $conf;

      if($conf['demo_mode'] == true) $app->error("Mail sending disabled in demo mode.");

      $app->uses('getconf,ispcmail');
      $mail_config = $app->getconf->get_global_config('mail');
      if($mail_config['smtp_enabled'] == 'y') {
         $mail_config['use_smtp'] = true;
         $app->ispcmail->setOptions($mail_config);
      }
      $app->ispcmail->setSender($from, $from_name);
      $app->ispcmail->setSubject($subject);
      $app->ispcmail->setMailText($text);

      if($filepath != '') {
         if(!file_exists($filepath)) $app->error("Mail attachement does not exist ".$filepath);
         $app->ispcmail->readAttachFile($filepath);
      }

      if($cc != '') $app->ispcmail->setHeader('Cc', $cc);
      if($bcc != '') $app->ispcmail->setHeader('Bcc', $bcc);

      $app->ispcmail->send($to);
      $app->ispcmail->finish();
		
      return true;
   }
	
   public function is_allowed_user($username, $check_id = true, $restrict_names = false) {
      global $app;
		
      if($username == 'root') return false;
      if($check_id && intval($this->getuid($username)) < $this->min_uid) return false;
		
      if($restrict_names == true && preg_match('/^web\d+$/', $username) == false) return false;
		
      return true;
   }
	
   public function is_allowed_group($groupname, $restrict_names = false) {
      global $app;
		
      if($groupname == 'root') return false;
      if(intval($this->getgid($groupname)) < $this->min_gid) return false;
		
      if($restrict_names == true && preg_match('/^client\d+$/', $groupname) == false) return false;
		
      return true;
   }
	
}

?>