如何保护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-上传文件夹 除了你已经做过的事情:

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

有一个人在上传到我的网络服务器PNG文件时,有了`<？php phpinfo();？>`插入到二进制数据中.此代码不可执行,但仍可通过`include()`函数访问.不幸的是,`getimagesize()`返回了非假值.我通过`$ img = imagecreatefrompng('vulnerability.png')重新保存所有上传的图像来管理这种情况; imagepng($ img,'safe.png');` (2认同)

Answer 2

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

不要将上传的文件保存在网络根目录中。为了更好地控制，上传的文件只能通过脚本访问。

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