Permalink
Browse files

MDL-25588 report_customlang: removed redundant safe path check

Only the $lang part of the the path comes from the browser and it is
checked in get_localpack_location() anyway.
  • Loading branch information...
mudrd8mz committed Dec 7, 2010
1 parent 2f698d2 commit b876a003b441dbb1be0f45e99761ad5dd0a86369
Showing with 0 additions and 3 deletions.
  1. +0 −3 admin/report/customlang/locallib.php
@@ -257,9 +257,6 @@ protected static function dump_strings($lang, $component, $strings) {
$packageinfo .= "\n * @subpackage $subpackage";
}
$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;
if (!is_dir(dirname($filepath))) {
check_dir_exists(dirname($filepath));

0 comments on commit b876a00

Please sign in to comment.