Improved input validation for settings that are acessible by the admin user.

Till Brehm

2014-08-01 e98a15838d8aed330ac7b1bb373b54524ea2a269

 interface/web/admin/form/server_config.tform.php

@@ -169,6 +169,9 @@
         'default' => '/var/backup',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'backup_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'backup_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -276,6 +279,9 @@
         'default' => '/home/vmail/[domain]/[localpart]/',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'maildir_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{5,128}$/',
                              'errmsg'=> 'maildir_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -287,6 +293,9 @@
         'default' => '/home/vmail/',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'homedir_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'homedir_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -310,6 +319,9 @@
         'default' => '5000',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'mailuser_uid_error_empty'),
                           1 => array('type' => 'RANGE',
                              'range' => '1999:',
                              'errmsg' => 'mailuser_uid_error_range'),
         ),
         'value' => '',
         'width' => '10',
@@ -321,6 +333,9 @@
         'default' => '5000',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'mailuser_gid_error_empty'),
                           1 => array('type' => 'RANGE',
                              'range' => '1999:',
                              'errmsg' => 'mailuser_gid_error_range'),
         ),
         'value' => '',
         'width' => '10',
@@ -332,6 +347,9 @@
         'default' => 'vmail',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'mailuser_name_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^(?!ispconfig|root)([a-zA-Z0-9]{1,20})$/',
                              'errmsg'=> 'mailuser_name_error_regex'),
         ),
         'value' => '',
         'width' => '10',
@@ -343,6 +361,9 @@
         'default' => 'vmail',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'mailuser_group_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^(?!ispconfig|root)([a-zA-Z0-9]{1,20})$/',
                              'errmsg'=> 'mailuser_group_name_error_regex'),
         ),
         'value' => '',
         'width' => '10',
@@ -452,6 +473,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'getmail_config_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'getmail_config_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -483,6 +507,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'website_basedir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'website_basedir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -494,6 +521,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'website_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{5,128}$/',
                              'errmsg'=> 'website_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -505,6 +535,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'website_symlinks_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]\:]{5,128}$/',
                              'errmsg'=> 'website_symlinks_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -538,6 +571,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'vhost_conf_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'vhost_conf_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -549,6 +585,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'vhost_conf_enabled_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'vhost_conf_enabled_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -560,6 +599,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'nginx_vhost_conf_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'nginx_vhost_conf_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -571,6 +613,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'nginx_vhost_conf_enabled_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'nginx_vhost_conf_enabled_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -579,6 +624,10 @@
      'CA_path' => array(
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'validators' => array(   0 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/]{0,128}$/',
                              'errmsg'=> 'ca_path_error_regex'),
         ),
         'default' => '',
         'value' => '',
         'width' => '40',
@@ -678,6 +727,13 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'apache_user_error_empty'),
               1 => array(
                     'type' => 'CUSTOM',
                     'class' => 'validate_systemuser',
                     'function' => 'check_sysuser',
                     'check_names' => false,
                     'errmsg' => 'invalid_apache_user_txt'
                  ),
         ),
         'value' => '',
         'width' => '40',
@@ -689,6 +745,13 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'apache_group_error_empty'),
               1 => array(
                     'type' => 'CUSTOM',
                     'class' => 'validate_systemuser',
                     'function' => 'check_sysgroup',
                     'check_names' => false,
                     'errmsg' => 'invalid_apache_group_txt'
                  ),
         ),
         'value' => '',
         'width' => '40',
@@ -717,6 +780,13 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'nginx_user_error_empty'),
                           1 => array(
                     'type' => 'CUSTOM',
                     'class' => 'validate_systemuser',
                     'function' => 'check_sysuser',
                     'check_names' => false,
                     'errmsg' => 'invalid_nginx_user_txt'
                  ),
         ),
         'value' => '',
         'width' => '40',
@@ -728,6 +798,13 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'nginx_group_error_empty'),
                           1 => array(
                     'type' => 'CUSTOM',
                     'class' => 'validate_systemuser',
                     'function' => 'check_sysgroup',
                     'check_names' => false,
                     'errmsg' => 'invalid_nginx_group_txt'
                  ),
         ),
         'value' => '',
         'width' => '40',
@@ -739,6 +816,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_ini_path_apache_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'php_ini_path_apache_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -750,6 +830,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_ini_path_cgi_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'php_ini_path_cgi_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -761,6 +844,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_fpm_init_script_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_]{1,128}$/',
                              'errmsg'=> 'php_fpm_init_script_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -772,6 +858,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_fpm_ini_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'php_fpm_ini_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -783,6 +872,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_fpm_pool_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'php_fpm_pool_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -792,7 +884,7 @@
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
         'validators' => array(0 => array('type' => 'ISPOSITIVE',
               'errmsg' => 'php_fpm_start_port_error_empty'),
         ),
         'value' => '',
@@ -805,6 +897,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_fpm_socket_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{5,128}$/',
                              'errmsg'=> 'php_fpm_socket_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -816,6 +911,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'php_open_basedir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/\]\[\:]{1,}$/',
                              'errmsg'=> 'php_open_basedir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -827,6 +925,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'nginx_cgi_socket_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'nginx_cgi_socket_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -884,6 +985,12 @@
      'awstats_data_dir' => array(
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'validators' => array(   0 => array('type' => 'NOTEMPTY',
                              'errmsg' => 'awstats_data_dir_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'awstats_data_dir_error_regex'),
         ),
         'default' => '',
         'value' => '',
         'width' => '40',
@@ -892,6 +999,12 @@
      'awstats_pl' => array(
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'validators' => array(   0 => array('type' => 'NOTEMPTY',
                              'errmsg' => 'awstats_pl_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'awstats_pl_error_regex'),
         ),
         'default' => '',
         'value' => '',
         'width' => '40',
@@ -900,6 +1013,12 @@
      'awstats_buildstaticpages_pl' => array(
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'validators' => array(   0 => array('type' => 'NOTEMPTY',
                              'errmsg' => 'awstats_buildstaticpages_pl_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'awstats_buildstaticpages_pl_error_regex'),
         ),
         'default' => '',
         'value' => '',
         'width' => '40',
@@ -925,6 +1044,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'bind_user_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^(?!ispconfig)([a-zA-Z0-9]{1,20})$/',
                              'errmsg'=> 'invalid_bind_user_txt'),
         ),
         'value' => '',
         'width' => '40',
@@ -936,6 +1058,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'bind_group_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^(?!ispconfig)([a-zA-Z0-9]{1,20})$/',
                              'errmsg'=> 'invalid_bind_group_txt'),
         ),
         'value' => '',
         'width' => '40',
@@ -947,6 +1072,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'bind_zonefiles_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'bind_zonefiles_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -958,6 +1086,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'named_conf_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'named_conf_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -969,6 +1100,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'named_conf_local_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'named_conf_local_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -994,6 +1128,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'fastcgi_starter_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{1,128}$/',
                              'errmsg'=> 'fastcgi_starter_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1005,6 +1142,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'fastcgi_starter_script_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'fastcgi_starter_script_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1016,6 +1156,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'fastcgi_alias_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'fastcgi_alias_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1027,6 +1170,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'fastcgi_phpini_path_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{1,128}$/',
                              'errmsg'=> 'fastcgi_phpini_path_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1036,7 +1182,7 @@
         'datatype' => 'VARCHAR',
         'formtype' => 'TEXT',
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
         'validators' => array(0 => array('type' => 'ISPOSITIVE',
               'errmsg' => 'fastcgi_children_error_empty'),
         ),
         'value' => '',
@@ -1063,6 +1209,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'fastcgi_bin_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{1,128}$/',
                              'errmsg'=> 'fastcgi_bin_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1097,6 +1246,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'jailkit_chroot_home_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/\[\]]{1,128}$/',
                              'errmsg'=> 'jailkit_chroot_home_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1108,6 +1260,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'jailkit_chroot_app_sections_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\-\_\ ]{1,128}$/',
                              'errmsg'=> 'jailkit_chroot_app_sections_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1119,6 +1274,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'jailkit_chroot_app_programs_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/\ ]{1,}$/',
                              'errmsg'=> 'jailkit_chroot_app_programs_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1130,6 +1288,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'jailkit_chroot_cron_programs_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\.\-\_\/\ ]{1,}$/',
                              'errmsg'=> 'jailkit_chroot_cron_programs_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1219,6 +1380,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'vlogger_config_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'vlogger_config_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1246,6 +1410,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'cron_init_script_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^[a-zA-Z0-9\-\_]{1,30}$/',
                              'errmsg'=> 'cron_init_script_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1257,6 +1424,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'crontab_dir_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'crontab_dir_error_regex'),
         ),
         'value' => '',
         'width' => '40',
@@ -1268,6 +1438,9 @@
         'default' => '',
         'validators' => array(0 => array('type' => 'NOTEMPTY',
               'errmsg' => 'cron_wget_error_empty'),
                           1 => array (    'type' => 'REGEX',
                              'regex' => '/^\/[a-zA-Z0-9\.\-\_\/]{1,128}$/',
                              'errmsg'=> 'cron_wget_error_regex'),
         ),
         'value' => '',
         'width' => '40',