[XE 기능개선] XE 1.11.6 이후 보안배치 #4

안내글 - https://xetown.com/thirdpartynews/1695431

변경된 파일 - https://github.com/YJSoft/xe-core/commit/5e613fa6ff7df8cc657eeb51839fc70654d864c2

1) classes/template/TemplateHandler.class.php  에서

protected function init   함수에서

if(substr($tpl_path, -1) != '/')

위에 추가

$tpl_path = preg_replace('/[\{\}\(\)\[\]<>\$\'"]/', '', $tpl_path);

2) modules/editor/editor.view.php 에서

function dispEditorSkinColorset() 함수에서

$skin = Context::get('skin');  

아래에 추가

if (!preg_match('/^[a-zA-Z0-9_-]+$/', $skin)) {

return $this->stop('msg_invalid_request');

}

$colorset = $skin_info->colorset;

를 아래걸로 변경

$colorset = $skin_info->colorset ?? null;

3)  modules/layout/layout.view.php   에서

public function dispLayoutPreviewWithModule()  함수에서

$layoutSrl = Context::get('layout_srl');

$module = Context::get('module_name');

$mid = Context::get('target_mid');

$skin = Context::get('skin');

$skinType = Context::get('skin_type');

부분을 아래걸로 변경

$layoutSrl = intval(Context::get('layout_srl'));

$module = preg_replace('/[^a-zA-Z0-9_]/', '', Context::get('module_name'));

$mid = preg_replace('/[^a-zA-Z0-9\/_-]/', '', Context::get('target_mid'));

$skin = preg_replace('/[^a-zA-Z0-9_-]/', '', Context::get('skin'));

$skinType = Context::get('skin_type') === 'M' ? 'M' : 'P';

4) modules/module/module.model.php  에서

function loadSkinInfo($path, $skin, $dir = 'skins') 함수에서

if(substr($path,-1)!='/') $path .= '/';    

를 아래걸로 변경

if(substr($path,-1) != '/') {

$path .= '/';

}

if(!preg_match('/^[a-zA-Z0-9_-]+$/', $skin)) {

return;

}

5) modules/poll/poll.controller.php 에서

function procPoll() 함수에서

$skin = Context::get('skin');

if(!$skin || !is_dir(_XE_PATH_ . 'modules/poll/skins/'.$skin)) $skin = 'default';

대신에 아래걸로 변경

$skin = Context::get('skin') ?: 'default';

if(!preg_match('/^[a-zA-Z0-9_-]+$/', $skin)) {

return $this->stop('msg_invalid_request');

}

if(!is_dir(_XE_PATH_ . 'modules/poll/skins/' . $skin)) {

$skin = 'default';

}

function procPollViewResult() 함수에서

$skin = Context::get('skin');

if(!$skin || !is_dir(_XE_PATH_ . 'modules/poll/skins/'.$skin)) $skin = 'default';

를 아래걸로 변경

$skin = Context::get('skin') ?: 'default';

if(!preg_match('/^[a-zA-Z0-9_-]+$/', $skin)) {

return $this->stop('msg_invalid_request');

}

if(!is_dir(_XE_PATH_ . 'modules/poll/skins/' . $skin)) {

$skin = 'default';

}

6) modules/poll/poll.model.php  에서

function getPollHtml  함수에서

$poll->style = $style;  

부분을 아래걸로 변경

$poll->style = preg_replace('/[^a-zA-Z0-9_-]/', '', $style);

$poll->stop_date = $output->data->stop_date;

아래에 추가

$skin = preg_replace('/[^a-zA-Z0-9_-]/', '', $skin);

function getPollResultHtml  함수에서

$poll->style = $style;  

부분을 아래걸로 변경

$poll->style = preg_replace('/[^a-zA-Z0-9_-]/', '', $style);

function getPollGetColorsetList 함수에서

$skin = Context::get('skin');

대신에 아래걸로 변경

$skin = Context::get('skin') ?: 'default';

if(!preg_match('/^[a-zA-Z0-9_-]+$/', $skin)) {

return $this->stop('msg_invalid_request');

}

if(!is_dir(_XE_PATH_ . 'modules/poll/skins/' . $skin)) {

$skin = 'default';

}