diff options
| author | Scott Ullrich <sullrich@pfsense.org> | 2010-04-27 22:08:53 -0400 |
|---|---|---|
| committer | Scott Ullrich <sullrich@pfsense.org> | 2010-04-27 22:08:53 -0400 |
| commit | 63f0333cf8e2832126c225f464e6613bb40693bd (patch) | |
| tree | ef4987658915f6d2841a6a4f8017772d9c787e77 /config | |
| parent | 89ad7b0561c17ef6ca35cf7d599d7b61790e0c2e (diff) | |
| download | pfsense-packages-63f0333cf8e2832126c225f464e6613bb40693bd.tar.gz pfsense-packages-63f0333cf8e2832126c225f464e6613bb40693bd.tar.bz2 pfsense-packages-63f0333cf8e2832126c225f464e6613bb40693bd.zip | |
Add input validation for the settings page
Diffstat (limited to 'config')
| -rw-r--r-- | config/varnish64/varnish.inc | 13 | ||||
| -rw-r--r-- | config/varnish64/varnish_settings.xml | 3 |
2 files changed, 16 insertions, 0 deletions
diff --git a/config/varnish64/varnish.inc b/config/varnish64/varnish.inc index 295533b9..d2d10e2c 100644 --- a/config/varnish64/varnish.inc +++ b/config/varnish64/varnish.inc @@ -31,6 +31,19 @@ */ /* ========================================================================== */ +function varnish_settings_post_validate($post, $input_errors) { + if($post['storagesize'] && !is_int($post['storagesize'])) + $input_errors[] = "A valid number is required for the field 'Storage type'"; + if($post['listeningport'] && !is_int($post['listeningport'])) + $input_errors[] = "A valid number is required for the field 'Listening port'"; + if($post['minworkers'] && !is_int($post['minworkers'])) + $input_errors[] = "A valid number is required for the field 'Minimum worker threads'"; + if($post['maxworkers'] && !is_int($post['maxworkers'])) + $input_errors[] = "A valid number is required for the field 'Maximum worker threads'"; + if($post['timeoutworkers'] && !is_int($post['timeoutworkers'])) + $input_errors[] = "A valid number is required for the field 'Worker thread timeout'"; +} + function varnish_lb_directors_post_validate($post, $input_errors) { if (preg_match("/[^a-zA-Z0-9]/", $post['directorname'])) $input_errors[] = "The directorname name must only contain the characters a-Z or 0-9"; diff --git a/config/varnish64/varnish_settings.xml b/config/varnish64/varnish_settings.xml index 82f1fa18..c3304c1a 100644 --- a/config/varnish64/varnish_settings.xml +++ b/config/varnish64/varnish_settings.xml @@ -120,4 +120,7 @@ sync_package_varnish(); varnish_start(); </custom_php_resync_config_command> + <custom_php_validation_command> + varnish_settings_post_validate($_POST, &$input_errors); + </custom_php_validation_command> </packagegui>
\ No newline at end of file |