# 漏洞加固报告

## 1. 漏洞点名称
文件上传功能的安全漏洞
该漏洞属于**文件上传类型漏洞**，可能导致攻击者上传恶意文件。关键特征包括：
- 缺乏对上传文件的严格校验（文件类型校验逻辑存在缺陷）。
- 未对服务器端文件路径和文件类型进行有效验证，可能被绕过。

---

## 2. 漏洞代码

### 前端代码 
function checkFile() {
    var file = document.getElementsByName('upload file')[0].value;
    if (file == null || file == "") {
        alert("请选择要上传的文件！");
        return false;
    }
    // 定义允许上传的文件类型
    var allow_ext = ".jpgl. pngl .gif";
    // 提取上传文件的类型
    var ext_name = file.substring(file.lastIndexOf("."));
    // 判断上传文件类型是否允许上传
    if (allow_ext.indexof(ext_name) == -1) {
        var errMsg = "该文件不允许上传，请上传" + allow_ext + "类型的文件，当前文件类型为：" + ext_name;
        alert(errMsg);
        return false;
    }
}


### 后端代码 
$is_upload = false;
$msg = null;
if (isset($_POST['submit'])) {
    if (file_exists(UPLOAD_PATH)) {
        $temp_file = $_FILES['upload_file']['tmp_name'];
        $img_path = UPLOAD_PATH . '/' . $_FILES['upload_file']['name'];
        if (move_uploaded_file($temp_file, $img_path)) {
            $is_upload = true;
        } else {
            $msg = '上传出错！';
        }
    } else {
        $msg = UPLOAD_PATH . '文件夹不存在，请手工创建！';
    }
}


---

## 3. 漏洞产生的危害
1. **文件类型校验缺陷**：
- 前端的文件校验机制仅通过字符串匹配检查文件扩展名，极易被篡改或绕过。
- 后端未对文件类型进行进一步验证，允许上传非图片类型的文件，如恶意脚本文件（`.php`、`.jsp` 等）。

2. **潜在风险**：
- 攻击者可能上传恶意脚本，通过访问恶意脚本获取对服务器的控制权。
- 可能泄露敏感信息或导致数据被篡改，甚至让服务器沦为攻击平台。

3. **业务影响**：
- 数据完整性和机密性受损。
- 攻击者可能实施勒索攻击，导致严重经济损失。

---

## 4. 漏洞产生的原因以及修复思路

### 漏洞产生的原因
1. **客户端校验缺陷**：客户端文件类型校验缺乏可靠性，容易被篡改。
2. **后端安全措施不足**：未严格验证上传文件的 MIME 类型、扩展名及文件内容一致性。

### 修复思路
1. **后端严格校验**：
- 使用 mime_content_type() 或类似函数验证文件真实的 MIME 类型。
- 限制允许的文件扩展名。
- 对文件内容进行校验，防止伪装攻击。

2. **前端增强校验**：
- 使用更加健壮的文件校验逻辑，但前端校验仅作为用户体验的辅助措施，不能替代后端校验。

3. **其他防御措施**：
- 上传目录设置不可执行权限，防止上传的文件被执行。
- 对上传文件进行重命名，防止路径预测攻击。

---

## 5. 漏洞修复后代码截图

### 修复后的前端代码 
function checkFile() {
    var file = document.getElementsByName('upload_file')[0].value;
    if (!file) {
        alert("请选择要上传的文件！");
        return false;
    }
    var allow_ext = [".jpg", ".png", ".gif"];
    var ext_name = file.substring(file.lastIndexOf(".")).toLowerCase();
    if (!allow_ext.includes(ext_name)) {
        alert(`该文件不允许上传，请上传以下类型的文件：${allow_ext.join(", ")}`);
        return false;
    }
    return true;
}


### 修复后的后端代码 
$is_upload = false;
$msg = null;
if (isset($_POST['submit'])) {
    if (file_exists(UPLOAD_PATH)) {
        $temp_file = $_FILES['upload_file']['tmp_name'];
        $file_info = finfo_open(FILEINFO_MIME_TYPE);
        $mime_type = finfo_file($file_info, $temp_file);
        finfo_close($file_info);

        // 允许的 MIME 类型和扩展名
        $allowed_types = ['image/jpeg', 'image/png', 'image/gif'];
        $allowed_ext = ['jpg', 'png', 'gif'];

        $file_ext = pathinfo($_FILES['upload_file']['name'], PATHINFO_EXTENSION);

        if (in_array($mime_type, $allowed_types) && in_array($file_ext, $allowed_ext)) {
            $new_file_name = uniqid() . '.' . $file_ext;
            $img_path = UPLOAD_PATH . '/' . $new_file_name;
            if (move_uploaded_file($temp_file, $img_path)) {
                $is_upload = true;
                $msg = '文件上传成功！';
            } else {
                $msg = '上传出错！';
            }
        } else {
            $msg = '文件类型不合法！';
        }
    } else {
        $msg = UPLOAD_PATH . '文件夹不存在，请手工创建！';
    }
}


---

## 关键词
#文件上传漏洞 #安全校验 #MIME类型验证 #恶意文件上传 #安全加固
 
 
Back to Top