Always use lowercase extension name (#5268)

Kunena · Mar 23, 2017 · 30324cd · 30324cd
1 parent 5cd9bae
commit 30324cd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/libraries/kunena/attachment/attachment.php b/src/libraries/kunena/attachment/attachment.php
@@ -458,8 +458,13 @@ function upload($key = 'kattachment', $catid = null)
 		$fileInput['name'] = preg_replace('/[[:space:]]/', '', $fileInput['name']);
 
 		$fileNameWithoutExt = JFile::stripExt($fileInput['name']);
+		$fileNameWithoutExt = strtolower($fileNameWithoutExt);
+
 		$fileExt = JFile::getExt($fileInput['name']);
+		$fileExt = strtolower($fileExt);
+
 		$fileNameWithExt = $fileInput['name'];
+		$fileNameWithExt = strtolower($fileNameWithExt);
 
 		if (file_exists($uploadBasePath . $fileInput['name']))
 		{

diff --git a/src/libraries/kunena/upload/upload.php b/src/libraries/kunena/upload/upload.php
@@ -561,6 +561,7 @@ public function upload($fileInput, $destination, $type = 'attachment')
 	{
 		$file = new stdClass;
 		$file->ext = JFile::getExt($fileInput['name']);
+		$file->ext = strtolower($file->ext);
 		$file->size = $fileInput['size'];
 		$config = KunenaFactory::getConfig();
 
@@ -587,7 +588,6 @@ public function upload($fileInput, $destination, $type = 'attachment')
 		if ($file->isAvatar)
 		{
 			$a = array('gif', 'jpeg', 'jpg', 'png');
-			$file->ext = strtolower($file->ext);
 
 			if (!in_array($file->ext, $a, true))
 			{