ThinkPHP系统常被挂马的代码

在使用 ThinkPHP 框架（无论是版本 3、5 还是 6）进行开发时，也有一些常见的代码结构和实践，如果不当，可能会导致安全漏洞，从而容易被攻击者利用。下面是一些示例代码及其潜在风险：

ThinkPHP 3.x 的常见安全问题 SQL 注入

$name = $_GET['name'];$result = M('User')->where("name='$name'")->select();

风险: 直接使用用户输入拼接 SQL 查询，容易导致 SQL 注入。不安全的文件上传

if (isset($_FILES['file'])) {    move_uploaded_file($_FILES['file']['tmp_name'], 'uploads/' . $_FILES['file']['name']);}

风险: 没有对文件类型或内容进行验证，恶意文件可能被上传。 未经验证的用户输入

$data = $_POST['data'];eval($data);

风险: 使用 eval() 执行用户的输入数据，极为危险。 ThinkPHP 5.x 的常见安全问题 SQL 注入

$name = input('get.name');$user = db('user')->where("name='$name'")->find();

风险: 直接拼接用户输入，导致 SQL 注入。 XSS 漏洞

return view('profile', ['name' => input('get.name')]);

风险: 如果 name 包含恶意脚本，可能导致 XSS 攻击。 不当的文件包含

$file = input('get.file');include($file);

风险: 允许用户控制包含的文件，可能导致远程代码执行。ThinkPHP 6.x 的常见安全问题 SQL 注入

$name = $request->get('name');$user = User::where("name='$name'")->find();

风险: 整合用户输入至查询中，存在 SQL 注入风险。不安全的文件操作

$filename = $request->get('file');file_put_contents($filename, 'Hello World');

风险: 允许用户输入文件名，可能导致覆盖重要文件。暴露调试信息

try {    // some code that may throw} catch (\Exception $e) {    return $e->getMessage(); // 直接返回错误信息}

风险: 直接返回错误信息可能暴露服务器路径或其他敏感信息。 

防范措施

1. 使用 ORM: 利用 ThinkPHP 提供的 ORM 功能，避免手动拼接 SQL。

2. 输入验证: 验证和过滤用户输入，确保安全性。

3. 文件上传: 校验文件类型、大小和内容，避免执行恶意代码。

4. 使用模板引擎: 通过模板引擎输出用户数据时，自动进行 HTML 转义。

5. 错误处理: 不向外界暴露具体的错误信息，仅记录日志，给用户友好的信息。

在开发过程中，遵循安全最佳实践抵御一些常见漏洞，可以有效降低应用被攻击的风险。

除了 SQL 注入之外，还有许多其他的代码和实践在使用 ThinkPHP 或其他 PHP 框架时具有安全隐患，可能导致被挂马或遭受攻击。以下是一些常见的示例和易受攻击的实践：

1. 不当的文件上传处理

if (isset($_FILES['file'])) {    move_uploaded_file($_FILES['file']['tmp_name'], 'uploads/' . $_FILES['file']['name']);}

风险: 该代码没有对文件的类型、大小和内容进行适当验证，攻击者可以上传可执行的恶意文件。

2. 不安全的反序列化

$data = $_POST['data'];$foo = unserialize($data);

风险: 如果用户可以控制序列化的数据，攻击者可以利用此途径执行恶意对象并导致代码执行。

3. 错误处理不当

try {    // 可能抛出异常的代码} catch (Exception $e) {    echo $e->getMessage(); // 直接输出异常信息}

风险: 直接输出异常信息可能泄露系统内部信息、文件路径或其他敏感数据。

4. 未过滤的外部输入

$content = $_POST['content'];file_put_contents('log.txt', $content);

风险: 允许用户随意写入文件，可能导致日志注入或文件覆盖。

5. 不当使用 include 或 require

$page = $_GET['page'];include($page);

风险: 直接包含用户输入的文件名，可能导致远程文件包含攻击（RFI）或本地文件包含攻击（LFI）。

6. CSRF（跨站请求伪造）

<form method="POST" action="/change-password">    <input type="text" name="new_password">    <input type="submit" value="Change Password"></form>

风险: 没有使用 CSRF 令牌，攻击者可以构造恶意请求，诱使用户在不知情的情况下更改敏感信息。

7. 文件权限设置不当

在上传文件或存储文件时，未正确设置文件权限（如使用 0777）。

move_uploaded_file($_FILES['file']['tmp_name'], 'uploads/' . $_FILES['file']['name']);chmod('uploads/' . $_FILES['file']['name'], 0777);

风险: 过高的文件权限设置可能使得攻击者能够执行或修改上传的文件。

8. JWT（JSON Web Token）不安全使用

$token = $_GET['token'];// 不进行验证和解析

风险: 没有验证和安全解析 JWT，攻击者可以伪造或篡改 token，实现未授权访问。

防范措施:

1. 限制文件上传: 验证文件类型、内容和大小，避免不必要的文件上传。

2. 避免反序列化: 对输入的数据进行严格控制，避免使用反序列化。

3. 错误处理: 使用日志记录异常而不是直接输出，同时确保重要信息不暴露。

4. 输入验证: 对所有用户输入进行严格验证。

5. 使用 CSRF 保护: 为敏感操作添加 CSRF 令牌。

6. 文件权限管理: 确保文件和目录的权限设置最小化。

7. 安全使用 JWT: 确保 JWT 的生成、解析和验证过程安全。

遵循这些最佳实践将大大提高应用程序的安全性，从而降低被挂马或其他攻击的风险。