moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-04 08:26:37 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-25588 report_customlang: removed redundant safe path check

Browse source 
Only the $lang part of the the path comes from the browser and it is
checked in get_localpack_location() anyway.
This commit is contained in:
David Mudrak 2010-12-07 06:08:51 +00:00
parent 2f698d2a0b
commit b876a003b4
1 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
admin/report/customlang/locallib.php
View file

@ -257,9 +257,6 @@ class report_customlang_utils {
$packageinfo .= "\n * @subpackage $subpackage"; $packageinfo .= "\n * @subpackage $subpackage";
} }
$filepath = self::get_localpack_location($lang); $filepath = self::get_localpack_location($lang);
if ($filepath !== clean_param($filepath, PARAM_SAFEPATH)) {
throw new coding_exception('Incorrect file location '.s($filepath));
}
$filepath = $filepath.'/'.$filename; $filepath = $filepath.'/'.$filename;
if (!is_dir(dirname($filepath))) { if (!is_dir(dirname($filepath))) {
check_dir_exists(dirname($filepath)); check_dir_exists(dirname($filepath));