如何保护PHP图像上载脚本免受攻击？

Question

我创建了（使用Stack 提供的脚本和一些帮助以及朋友的一些帮助;我对PHP知之甚少）一个简单的页面，用于本地非营利性出版物，人们可以上传照片。

我对安全性不是很好（基于无知，而不是故意疏忽）但我采取了以下步骤来保护这个页面：

•PHP脚本设置为仅接受.jpg，.png和.tif文件进行上传;
•保存表单内容的子文件夹具有设置为700的权限，并且保存上传照片的子文件夹具有设置为700的权限;
•根据文档，我的主机具有以下配置，以确保只有.php文件以.php运行：

<FilesMatch \.php$>
    SetHandler php52-fcgi
</FilesMatch>

•我在相关（主要和保存的内容）文件夹中放置了.htaccess文件：

RemoveHandler .php
RemoveHandler .inc
RemoveHandler .pl
RemoveHandler .cgi
RemoveHandler .py
RemoveHandler .fcgi

然而，一夜之间，有人找到了这个测试页面，并提交了一个看起来非常良性的测试信息和小.jpg。 这是一个私人测试页面，其中包含一个非直观的网址，只有我和其他三个人都知道; 其他人都没有发过这个测试。

这显然让我担心会发生一些奇怪的事情，我担心我对安全性的了解不足以确保这个页面是安全的。

有什么明显的东西让我失踪吗？

Answer 1

在处理上传时，您应该记住，您可以在$ _FILES数组中找到的所有数据都是伪造的。 它正在通过HTTP进行传输，因此将image / jpg mime提供给可执行文件非常容易。

1-检查真正的哑剧

PHP附带了一些功能来检查文件的真实mime。 为此你应该使用fileinfo

$finfo = new finfo(FILEINFO_MIME, "/usr/share/misc/magic");
$filename = "/var/tmp/afile.jpg";
echo $finfo->file($filename);

2-检查图像的属性

您显然只想上传图片，因此收到的文件必须有宽度和高度：

使用getImageSize（）获取有关图像的所有必需信息。 如果返回false，则该文件可能不是图像，您可以将其删除。 getImageSize也可以给你一个mime类型，但我不知道它是否可以信任。

2.5-重新处理图像

正如user628405所建议的那样，用GD重新处理图像可能是更安全的事情。

$img = imagecreatefrompng('vulnerable.png'); 
imagepng($img, 'safe.png');

显然它必须根据图像类型进行调整。 在php文档中查看所有imagecreatefrom *。

3-上传文件夹除了你已经做过的事情：

确保您的上传文件夹不在网上。 验证上载的文件，然后根据需要将其移动到其他文件夹并重命名该文件。 它将阻止黑客执行恶意文件（如果网址无法访问，则无法执行）。

Answer 2

不要依赖客户端的任何数据，包括内容类型！

不要将上载的文件保存在Web根目录中。 只能通过脚本访问上传的文件，以便更好地控制。

不要使用原始文件名和扩展名保存上传的文件！ 将此数据存储在数据库中以便以后检索。

Answer 3

你可以检查文件的MIME类型，但是不要担心，只要你的php处理程序只能执行.php文件而你正在处理不保存脚本中上传的.php文件，你不会暴露任何文件安全漏洞。

这对于.php文件以及服务器上安装的任何其他服务器端脚本语言当然有效。

更好的想法是保留您要接受的扩展名以保存在文件系统上。

Answer 4

我会忽略MIME类型和传入文件的文件扩展名。 这些可以伪造。

如果您要沿着那条大道走下去，请将这些文件存储在目录中。

确保该目录仅用于图像（音乐），然后通过查看文件格式获取脚本以在其上放置正确的扩展名。

还要确保该目录不能执行PHP（或其他任何东西）。

这将保证您的安全。