检查安全图像上传答案

【问题标题】：Check secure image upload检查安全图像上传
【发布时间】：2015-01-25 14:23:50
【问题描述】：

我想从浏览器窗口上传图片到服务器。我正在使用此功能进行图像上传。图像由 GD 库上传。
您对此代码有何看法？
我想知道是否还有办法上传恶意文件？

<?php
function imageUpload($name, $thumb = false, $max_size = 5242880, $height = null, $width = null, $T__height = 100, $T__width = 100)
{
    if(!isset($_FILES[$name]))
    {
        return false;
    }

    $allowedTypes = array('image/gif', 'image/jpeg', 'image/png', 'image/wbmp');
    $image = &$_FILES[$name];

    if ($image['error'] == 0 && in_array($image['type'], $allowedTypes) && $image['size'] <= $max_size)
    {
        $in = '';
        switch ($image['type'])
        {
            case 'image/gif':
                $in = 'imagecreatefromgif';
                break;
            case 'image/jpeg':
                $in = 'imagecreatefromjpeg';
                break;
            case 'image/png':
                $in = 'imagecreatefrompng';
                break;
            case 'image/wbmp':
                $in = 'imagecreatefromwbmp';
                break;
        }

        if ($in == '')
        {
            return false;
        }

        $src = $in($image['tmp_name']);
        $height = ($height == null || $height <= 0 ? imagesy($src) : $height);
        $width = ($width == null || $width <= 0 ? imagesx($src) : $width);

        $dst = imagecreatetruecolor($width, $height);
        imagecopyresized($dst, $src, 0, 0, 0, 0, $width, $height, imagesx($src), imagesy($src));

        $fileName = '';
        do
        {
            $fileName = makeHash(mt_rand(0, mt_getrandmax()) . microtime(true), $image['tmp_name']);
        }
        while(file_exists('/image/' . $fileName . '.jpg') || ($thumb && file_exists('/thumb/' . $fileName . '.jpg')));
        if ($fileName == '')
        {
            return false;
        }

        imagejpeg($dst, '/image/' . $fileName . '.jpg', 75);

        if ($thumb)
        {
            $dst = imagecreatetruecolor($T__width, $T__height);
            imagecopyresized($dst, $src, 0, 0, 0, 0, $T__width, $T__height, imagesx($src), imagesy($src));
            imagejpeg($dst, '/thumb/' . $fileName . '.jpg', 75);
        }

        imagedestroy($src);
        imagedestroy($dst);

        return $fileName . '.jpg';
    }
}
?>

【问题讨论】：

我投票决定将此问题作为离题结束，因为它适合 codereview.stackexchange.com，而不是 Stack Overflow。

标签： php security upload image-upload

【解决方案1】：

首先，任何类型的安全机制都不应依赖 mimetypes，因为攻击者可以完全控制该字段（他们可以上传一个图像/gif 类型的 .php 文件，该文件将通过这些检查） .然后，如果它们在文件名%00 或\00 中包含一个空字节，则可以绕过像evil.php\00.jpg 这样为图像添加.jpg 扩展名的条件，所以当文件到达磁盘时将只是 evil.php。但是，如果不查看所有函数，就很难完全评估您的代码，因为根据它们所做的一切可能会打开漏洞。（例如，不清楚它们的文件存储在哪里、调用它们的内容、加载它们等）。

【讨论】：