Fixes #5188 Unclear error when geticon.php gets wrong information

829ffd16 · Côme Chilliet · ada0b00d · 829ffd16
Commit 829ffd16 authored 8 years ago by Côme Chilliet
Hide whitespace changes
Inline Side-by-side

Showing

with 4 additions and 0 deletions
+4 -0
--- a/html/geticon.php
+++ b/html/geticon.php
@@ -36,6 +36,10 @@ if (session::global_is_set('config')) {
 }
 IconTheme::$default_theme = 'breezy';
 IconTheme::$extensions    = array('png');
+if (!isset($_GET['context']) || !isset($_GET['icon']) || !isset($_GET['size'])) {
+  trigger_error('Missing information in query string: '.$_SERVER['QUERY_STRING']);
+  exit;
+}
 $src    = IconTheme::findThemeIcon($theme, $_GET['context'], $_GET['icon'], $_GET['size']);

 header("Content-Type: image/png");