通达OA_文件包含漏洞

一、漏洞描述

复现靶场:通达2017版

漏洞文件:td\webroot\ispirit\interface\gateway.php

漏洞地址: /mac/gateway.php

漏洞POC:json={"url":"/general/.../.../mysql5/my.ini"}

利用工具:https://github.com/dioos886/Tongda_File_contains

二、原始gateway.php文件代码

php 复制代码
<?php

ob_start();
include_once "inc/session.php";
include_once "inc/conn.php";
include_once "inc/utility_org.php";

if ($P != "") {
	if (preg_match("/[^a-z0-9;]+/i", $P)) {
		echo _("非法参数");
		exit();
	}

	session_id($P);
	session_start();
	session_write_close();
	if (($_SESSION["LOGIN_USER_ID"] == "") || ($_SESSION["LOGIN_UID"] == "")) {
		echo _("RELOGIN");
		exit();
	}
}

if ($json) {
	$json = stripcslashes($json);
	$json = (array) json_decode($json);

	foreach ($json as $key => $val ) {
		if ($key == "data") {
			$val = (array) $val;

			foreach ($val as $keys => $value ) {
				$keys = $value;
			}
		}

		if ($key == "url") {
			$url = $val;
		}
	}

	if ($url != "") {
		if (substr($url, 0, 1) == "/") {
			$url = substr($url, 1);
		}

		if ((strpos($url, "general/") !== false) || (strpos($url, "ispirit/") !== false) || (strpos($url, "module/") !== false)) {
			include_once $url;
		}
	}

	exit();
}

?>

根据PHP代码存在以下问题

三、文件包含漏洞

php 复制代码
if ($url != "") {
    if (substr($url, 0, 1) == "/") {
        $url = substr($url, 1);
    }

    if ((strpos($url, "general/") !== false) || (strpos($url, "ispirit/") !== false) || (strpos($url, "module/") !== false)) {
        include_once $url;
    }
}

这部分代码允许从外部传递的JSON数据中提取$url,然后通过include_once包含该URL。这样的代码非常危险,因为攻击者可以构造恶意的URL来包含任意文件。解决这个问题的方法是严格限制可以包含的文件,或者使用白名单来验证文件路径。

文件包含代码部分

这段PHP代码的主要目的是包含文件,但它存在安全问题,因为未对输入进行充分验证和过滤,导致文件包含漏洞。让我详细解释这段代码的运行流程和作用:

  1. 条件检查 :代码的第一行是一个条件检查,它检查变量 $url 是否为空字符串。如果 $url 不为空(即有值),则会继续执行后续的代码。

  2. URL 规范化 :接下来,代码使用 substr 函数检查 $url 的第一个字符是否是斜杠 /。如果 $url 的第一个字符是斜杠,代码将使用 substr 函数将斜杠去掉,以规范化 $url,确保它不以斜杠开头。这一步的目的是为了确保 $url 的格式符合预期。

  3. 包含文件 :最后,代码使用 strpos 函数检查 $url 是否包含子字符串 "general/"、"ispirit/" 或 "module/" 中的任何一个。如果 $url 中包含任何一个子字符串,代码将使用 include_once 函数来包含 $url 指定的文件。

    • 如果 $url 中包含 "general/",则会尝试包含名为 "general/" 的文件。
    • 如果 $url 中包含 "ispirit/",则会尝试包含名为 "ispirit/" 的文件。
    • 如果 $url 中包含 "module/",则会尝试包含名为 "module/" 的文件。

需要注意的是,如果 $url 中包含多个子字符串(例如 "general/" 和 "ispirit/"),那么将尝试依次包含这些文件,而不会同时包含多个文件。

这段代码的主要问题在于它没有对 $url 进行足够的输入验证和过滤,因此存在文件包含漏洞。攻击者可以构造恶意的 $url,以尝试包含系统中的敏感文件。为了增加安全性,应该对传入的 $url 进行严格的验证和白名单控制,确保只能包含可信任的文件,同时避免包含恶意文件。

gateway.php文件中的Json未对输入进行充分过滤和验证

php 复制代码
$json = stripcslashes($json);
$json = (array) json_decode($json);

foreach ($json as $key => $val ) {
    if ($key == "data") {
        $val = (array) $val;

        foreach ($val as $keys => $value ) {
            $keys = $value;
        }
    }

    if ($key == "url") {
        $url = $val;
    }
}

这段代码对传入的JSON数据进行了处理,但没有充分验证和过滤输入。恶意用户可以发送恶意JSON数据,导致不可预料的问题。建议对输入数据进行严格的验证和过滤,以防止恶意输入。

相关推荐
合作小小程序员小小店21 小时前
Web渗透之一句话木马
web安全·网络安全·安全威胁分析·安全架构·安全性测试
mit6.8249 天前
[code-review] 文件过滤逻辑 | 范围管理器
人工智能·代码复审
帅次10 天前
系统分析师-软件工程-软件开发环境与工具&CMM&CMMI&软件重用和再工程
性能优化·软件工程·软件构建·需求分析·规格说明书·代码复审·极限编程
合作小小程序员小小店22 天前
机器学习介绍
人工智能·python·机器学习·scikit-learn·安全威胁分析
mit6.82423 天前
[code-review] 日志机制 | `LOG_LEVEL`
人工智能·chatgpt·代码复审
mit6.82423 天前
[code-review] AI聊天接口 | 语言模型通信器
人工智能·语言模型·代码复审
mit6.82423 天前
[code-review] 部署配置 | Docker+PM2 | AWS Lambda | Vercel+边缘函数
代码复审
mit6.82424 天前
[code-review] Probot应用核心 | 审查协调器
代码复审
合作小小程序员小小店1 个月前
web渗透PHP反序列化漏洞
前端·网络协议·web安全·网络安全·安全威胁分析
合作小小程序员小小店1 个月前
挖漏洞三步走
python·网络协议·web安全·网络安全·安全威胁分析