mirror of
https://github.com/Piwigo/Piwigo.git
synced 2026-03-28 17:42:57 +01:00
fixes #847, CVE-2018-5692 protect a few user input variables
This commit is contained in:
@@ -42,6 +42,7 @@ trigger_notify('loc_begin_admin');
|
||||
check_status(ACCESS_ADMINISTRATOR);
|
||||
|
||||
check_input_parameter('page', $_GET, false, '/^[a-zA-Z\d_-]+$/');
|
||||
check_input_parameter('section', $_GET, false, '/^[a-z]+[a-z_\/-]*(\.php)?$/i');
|
||||
|
||||
// +-----------------------------------------------------------------------+
|
||||
// | Direct actions |
|
||||
|
||||
@@ -42,6 +42,7 @@ include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
|
||||
check_status(ACCESS_ADMINISTRATOR);
|
||||
|
||||
check_input_parameter('selection', $_POST, true, PATTERN_ID);
|
||||
check_input_parameter('display', $_REQUEST, false, '/^(\d+|all)$/');
|
||||
|
||||
// +-----------------------------------------------------------------------+
|
||||
// | specific actions |
|
||||
|
||||
@@ -42,6 +42,8 @@ include_once(PHPWG_ROOT_PATH.'include/functions_mail.inc.php');
|
||||
// +-----------------------------------------------------------------------+
|
||||
check_status(ACCESS_ADMINISTRATOR);
|
||||
|
||||
check_input_parameter('mode', $_GET, false, '/^(param|subscribe|send)$/');
|
||||
|
||||
// +-----------------------------------------------------------------------+
|
||||
// | Initialization |
|
||||
// +-----------------------------------------------------------------------+
|
||||
|
||||
@@ -33,6 +33,7 @@ include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
|
||||
// +-----------------------------------------------------------------------+
|
||||
check_status(ACCESS_ADMINISTRATOR);
|
||||
|
||||
check_input_parameter('display', $_GET, false, PATTERN_ID);
|
||||
|
||||
include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
|
||||
$tabsheet = new tabsheet();
|
||||
|
||||
@@ -37,7 +37,10 @@ STEP:
|
||||
3 = upgrade on different branch
|
||||
*/
|
||||
$step = isset($_GET['step']) ? $_GET['step'] : 0;
|
||||
|
||||
check_input_parameter('to', $_GET, false, '/^\d+\.\d+\.\d+$/');
|
||||
$upgrade_to = isset($_GET['to']) ? $_GET['to'] : '';
|
||||
|
||||
$updates = new updates();
|
||||
|
||||
// +-----------------------------------------------------------------------+
|
||||
|
||||
Reference in New Issue
Block a user