Static content: Fix check whether an image exists
This commit is contained in:
parent
e7e1b1a98e
commit
e5e64ab654
|
@ -59,13 +59,14 @@ class StaticController extends Controller
|
||||||
*/
|
*/
|
||||||
public function imgAction()
|
public function imgAction()
|
||||||
{
|
{
|
||||||
|
// TODO(el): I think this action only retrieves images from modules
|
||||||
$module = $this->_getParam('module_name');
|
$module = $this->_getParam('module_name');
|
||||||
$file = $this->_getParam('file');
|
$file = $this->_getParam('file');
|
||||||
$basedir = Icinga::app()->getModuleManager()->getModule($module)->getBaseDir();
|
$basedir = Icinga::app()->getModuleManager()->getModule($module)->getBaseDir();
|
||||||
|
|
||||||
$filePath = realpath($basedir . '/public/img/' . $file);
|
$filePath = realpath($basedir . '/public/img/' . $file);
|
||||||
|
|
||||||
if (! $filePath || strpos($filePath, $basedir) !== 0) {
|
if ($filePath === false) {
|
||||||
$this->httpNotFound('%s does not exist', $filePath);
|
$this->httpNotFound('%s does not exist', $filePath);
|
||||||
}
|
}
|
||||||
if (preg_match('/\.([a-z]+)$/i', $file, $m)) {
|
if (preg_match('/\.([a-z]+)$/i', $file, $m)) {
|
||||||
|
|
Loading…
Reference in New Issue