Better way of cleaning the file extension to prevent XSS attacks

This commit is contained in:
Sebastiaan Janssen
2017-01-05 11:51:54 +01:00
parent 69d6609a3b
commit e6b1ecb851

View File

@@ -525,7 +525,8 @@ namespace Umbraco.Web.Editors
foreach (var file in result.FileData)
{
var fileName = file.Headers.ContentDisposition.FileName.Trim(new[] { '\"' }).TrimEnd();
var ext = fileName.Substring(fileName.LastIndexOf('.') + 1).ToLower();
var safeFileName = fileName.ToSafeFileName();
var ext = safeFileName.Substring(safeFileName.LastIndexOf('.') + 1).ToLower();
if (UmbracoConfig.For.UmbracoSettings().Content.DisallowedUploadFiles.Contains(ext) == false)
{